[PATCH] Don't do FoldCmpLoadFromIndexedGlobal for non inbounds GEPs

Eli Friedman eli.friedman at gmail.com
Thu Aug 15 15:11:58 PDT 2013


I'm not sure all of the noinbounds tests are actually useful given none of
them trigger the transformation in question anymore.  Please look over them
before you commit.

Otherwise LGTM.

-Eli


On Wed, Aug 14, 2013 at 4:14 PM, Matt Arsenault
<Matthew.Arsenault at amd.com>wrote:

> Hi eli.friedman,
>
> http://llvm-reviews.chandlerc.com/D1261
>
> CHANGE SINCE LAST DIFF
>   http://llvm-reviews.chandlerc.com/D1261?vs=3202&id=3485#toc
>
> Files:
>   lib/Transforms/InstCombine/InstCombineCompares.cpp
>   test/Transforms/InstCombine/getelementptr.ll
>   test/Transforms/InstCombine/icmp.ll
>   test/Transforms/InstCombine/load-cmp.ll
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130815/0377f43e/attachment.html>


More information about the llvm-commits mailing list