[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