On 12/23/2017 03:28 AM, Jakub Jelinek wrote:
Hi!
Recently I've changed replace_placeholders to walk trees without duplicates
to avoid compile time explosion with lots of nested SAVE_EXPRs.
The problem as the following testcase shows is that it also prevents
replacement of PLACEHOLDER_EXPRs we want to replace, if the same
PLACEHOLDER_EXPR appears multiple time in the expression.
The following patch changes the walk_tree not to use pset itself, but rather
stops walking children if we've seen some tree already.
Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk/7.3?
ok, thanks.
nathan
--
Nathan Sidwell
