[llvm-commits] [llvm] r154673 - /llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp

[Duncan Sands]

Hi Hal,

>> Did you experiment to see if it was better to run EarlyCSE before
>> rather than
>> after instcombine?
>
> No, but I'd be very surprised if it was because instcombine cleans up all of the shuffles and insert/extract pairs. Nevertheless, it is worth a try, I'll let you know.

if earlyCSE unifies lots of instructions it might make instcombine run faster.
So it could be an interesting choice if (1) this is actually true, and (2) it
optimizes equally well.

Ciao, Duncan.