[PATCH] D81827: [CGP] Convert phi types

[Dave Green via Phabricator via llvm-commits]

dmgreen added a comment.

Compile time? 2x? That sounds like a lot! It must be hitting some degenerate cases much more than I expected..

Do you have a reproducer? I don't think any of my testing has turned up anything that big. (Although I'm worried now it was lost in the noise of the measurements. 2x should be easy enough to see though). http://llvm-compile-time-tracker.com/ doesn't show anything obvious either. There's perhaps a little movement, but nothing like 2x. Everything is +-0.05% and seems to go largely away with the next build.

I will try to run some more benchmarks, but it would be good to see what is causing a 2x change, exactly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81827