[PATCH] D65148: [SimplifyCFG] Bump phi-node-folding-threshold from 2 to 3

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 30 12:07:24 PDT 2019


lebedev.ri added a comment.

In D65148#1603922 <https://reviews.llvm.org/D65148#1603922>, @dmgreen wrote:

> ...


Hmm, i never got this mail, sorry for not noticing this reply. Thank you for looking into it!
I believe this shows the change: https://godbolt.org/z/Mb6T3R
That indeed looks not great. But i think i'm missing something.
That replacement does not look correct to me: https://rise4fun.com/Alive/CNns
Did you run the tests, too, or only benchmarks?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65148/new/

https://reviews.llvm.org/D65148





More information about the llvm-commits mailing list