[PATCH] D11051: Extend LICM to hoist loop invariant GEP out
hfinkel at anl.gov
hfinkel at anl.gov
Tue Jul 28 11:09:41 PDT 2015
hfinkel added a comment.
In http://reviews.llvm.org/D11051#213677, @atrick wrote:
> If you need to do this transformation is cases where the inbounds flag will be dropped it seems best to do it late in the pipeline (maybe even after LSR) as a target-controlled optimization. Arranging for the hoisting to happen seems like the easy part of this problem.
I agree. Also, as it turns out, both NVPTX and PowerPC run LICM late in the pipeline, and AArch64 can as well when -aarch64-gep-opt is enabled (PowerPC also runs it under the control of -ppc-gep-opt, but unlike AArch64, it is currently on by default).
> Out of curiosity, I wonder if the LSR fix here http://reviews.llvm.org/D11212 has any impact on this test case.
Repository:
rL LLVM
http://reviews.llvm.org/D11051
More information about the llvm-commits
mailing list