[PATCH] [InstCombine] Don't fold a GEP into itself through a PHI node

Nick Lewycky nicholas at mxc.ca
Wed Mar 18 20:30:41 PDT 2015


Daniel Jasper wrote:
> Ping?

     if (!Op2 || Op1->getNumOperands() != Op2->getNumOperands())
       return nullptr;
​
+   // As for Op1 above, don't try to fold a GEP into itself.
+   if (Op2 == &GEP)
+     return nullptr;

Consider folding that into the check above.

LGTM

> http://reviews.llvm.org/D8245
>
> EMAIL PREFERENCES
>    http://reviews.llvm.org/settings/panel/emailpreferences/
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>





More information about the llvm-commits mailing list