[PATCH] D48604: [AMDGPU] Enable LICM in the BE pipeline

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec added inline comments.


================
Comment at: lib/Target/AMDGPU/AMDGPUTargetMachine.cpp:642
   if (TM.getOptLevel() > CodeGenOpt::None) {
+    addPass(createLICMPass());
     addPass(createInferAddressSpacesPass());
----------------
arsenm wrote:
> Why here? Does this trigger a dominator tree recalculation? If so can it be moved somewhere else where it's already available?
I can move it into the top of addStraightLineScalarOptimizationPasses(). This will be after the SROA, so DT will exist.


https://reviews.llvm.org/D48604