Allow varying ranges of unknown types in irange::verify_range [PR109711]
The old legacy code allowed building ranges of unknown types so passes like IPA could build and propagate VARYING. For now it's easiest to allow the old behavior, it's not like you can do anything with these ranges except build them and copy them. Eventually we should convert all users of set_varying() to use supported types. I will address this in my upcoming IPA work. PR tree-optimization/109711 gcc/ChangeLog: * value-range.cc (irange::verify_range): Allow types of error_mark_node.
This commit is contained in:
parent
cef0c0bb13
commit
2b8a27634f
3 changed files with 47 additions and 0 deletions
16
gcc/testsuite/gcc.dg/tree-ssa/pr109711-1.c
Normal file
16
gcc/testsuite/gcc.dg/tree-ssa/pr109711-1.c
Normal file
|
@ -0,0 +1,16 @@
|
|||
// { dg-do compile }
|
||||
// { dg-options "-O2" }
|
||||
|
||||
void lspf2lpc();
|
||||
|
||||
void interpolate_lpc(int subframe_num) {
|
||||
float weight = 0.25 * subframe_num + 1;
|
||||
if (weight)
|
||||
lspf2lpc();
|
||||
}
|
||||
|
||||
void qcelp_decode_frame() {
|
||||
int i;
|
||||
for (;; i++)
|
||||
interpolate_lpc(i);
|
||||
}
|
24
gcc/testsuite/gcc.dg/tree-ssa/pr109711-2.c
Normal file
24
gcc/testsuite/gcc.dg/tree-ssa/pr109711-2.c
Normal file
|
@ -0,0 +1,24 @@
|
|||
// { dg-do compile }
|
||||
// { dg-options "-O2" }
|
||||
|
||||
void lspf2lpc();
|
||||
|
||||
int interpolate_lpc_q_0;
|
||||
|
||||
void
|
||||
interpolate_lpc(int subframe_num) {
|
||||
float weight;
|
||||
if (interpolate_lpc_q_0)
|
||||
weight = subframe_num;
|
||||
else
|
||||
weight = 1.0;
|
||||
if (weight != 1.0)
|
||||
lspf2lpc();
|
||||
}
|
||||
|
||||
void
|
||||
qcelp_decode_frame() {
|
||||
int i;
|
||||
for (;; i++)
|
||||
interpolate_lpc(i);
|
||||
}
|
|
@ -1057,6 +1057,13 @@ irange::verify_range ()
|
|||
return;
|
||||
}
|
||||
gcc_checking_assert (m_num_ranges <= m_max_ranges);
|
||||
|
||||
// Legacy allowed these to represent VARYING for unknown types.
|
||||
// Leave this in for now, until all users are converted. Eventually
|
||||
// we should abort in set_varying.
|
||||
if (m_kind == VR_VARYING && m_type == error_mark_node)
|
||||
return;
|
||||
|
||||
unsigned prec = TYPE_PRECISION (m_type);
|
||||
if (m_kind == VR_VARYING)
|
||||
{
|
||||
|
|
Loading…
Add table
Reference in a new issue