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

2018-01-02 Thread 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 wa

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

2017-12-23 Thread Jakub Jelinek
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 app