aboutsummaryrefslogtreecommitdiff
path: root/gcc/tree.cc
diff options
context:
space:
mode:
authorAndrew Pinski <apinski@marvell.com>2023-09-16 15:19:58 -0700
committerAndrew Pinski <apinski@marvell.com>2023-09-18 13:45:30 -0700
commit951d3c191d01440ad54415f683437770b0c957e4 (patch)
treef3ddd452db2b9e8bc7f0d1f3f5aa2e48d1431729 /gcc/tree.cc
parent80968d5f4683ffb50dbe8051d10f754d5fd00dfb (diff)
downloadgcc-951d3c191d01440ad54415f683437770b0c957e4.zip
gcc-951d3c191d01440ad54415f683437770b0c957e4.tar.gz
gcc-951d3c191d01440ad54415f683437770b0c957e4.tar.bz2
MATCH: Avoid recursive zero_one_valued_p for conversions
So when VN finds a name which has a nop conversion, it says both names are equivalent to each other and the valuaization function for one will return the other. This normally does not cause any issues as there is no recursive matches. But after r14-4038-gb975c0dc3be285, there was one added. So we would do an infinite recursion on the match and never finish. This fixes the issue (and adds a comment in match.pd) by for converts just handle one level instead of being recursive always. OK? Bootstrapped and tested on x86_64-linux-gnu with no regressions. Note the testcase was reduced from tree-ssa-loop-niter.cc and then changed slightly into C rather than C++ but it still needs exceptions turned on get the IR that VN would produce this equivalence relationship going on. Also had to turn off early inline to force put to be inlined later. PR tree-optimization/111435 gcc/ChangeLog: * match.pd (zero_one_valued_p): Don't do recursion on converts. gcc/testsuite/ChangeLog: * gcc.c-torture/compile/pr111435-1.c: New test.
Diffstat (limited to 'gcc/tree.cc')
0 files changed, 0 insertions, 0 deletions