Re: [C++ PATCH] Fix compile time hog in replace_placeholders (PR sanitizer/81929)

2017-09-22 Thread Jason Merrill
OK. On Thu, Sep 14, 2017 at 4:12 PM, Jakub Jelinek wrote: > Hi! > > When the expression replace_placeholders is called on contains > many SAVE_EXPRs that appear more than once in the tree, we hang walking them > over and over again, while it is sufficient to just walk it without > duplicates (not

[C++ PATCH] Fix compile time hog in replace_placeholders (PR sanitizer/81929)

2017-09-14 Thread Jakub Jelinek
Hi! When the expression replace_placeholders is called on contains many SAVE_EXPRs that appear more than once in the tree, we hang walking them over and over again, while it is sufficient to just walk it without duplicates (not using cp_walk_tree_without_duplicates, because the callback can cp_wal