[C++ PATCH] Fix replace_placeholders (PR c++/83556)

[Nathan Sidwell]

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