[PATCH] D22652: GVH-hoist: only clone GEPs (PR28606)

Eli Friedman via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 21 16:29:30 PDT 2016

Previous message: [PATCH] D22652: GVH-hoist: only clone GEPs (PR28606)
Next message: [PATCH] D22652: GVH-hoist: only clone GEPs (PR28606)
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

eli.friedman added a comment.

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.)


https://reviews.llvm.org/D22652

Previous message: [PATCH] D22652: GVH-hoist: only clone GEPs (PR28606)
Next message: [PATCH] D22652: GVH-hoist: only clone GEPs (PR28606)
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list