[PATCH] D22652: GVH-hoist: only clone GEPs (PR28606)
Sebastian Pop via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 21 16:40:06 PDT 2016
sebpop added a comment.
In https://reviews.llvm.org/D22652#492205, @eli.friedman wrote:
> You're missing an allOperandsAvailable call if Val is a GEP. (I'm not sure why you're even trying to special-case GEPs here.)
GEPs are hoisted only when the ld or st can be moved.
I added that check.
Repository:
rL LLVM
https://reviews.llvm.org/D22652
More information about the llvm-commits
mailing list