[Patch] PHI->GEP->PHI combine, take 2

[David Majnemer]

> GetElementPtrInst *NewGEP = dyn_cast<GetElementPtrInst>(Op1->clone());

This could just be a cast<GetElementPtrInst>(Op1->clone()).


On Tue, May 27, 2014 at 3:58 PM, Chandler Carruth <chandlerc at google.com>wrote:

>
> On Tue, May 27, 2014 at 3:07 PM, Louis Gerbarg <lgg at apple.com> wrote:
>
>> I would appreciate it if someone who is on a platform that supports
>> sanitizers could it a whirl.
>
>
> If you've tested it, and it's already been reviewed (which I think it had)
> please feel free to directly commit. If you'd like pre-commit eyes on the
> updated bits let me know and I'll look. But we have bots to check things
> like sanitizers that don't work on all platforms, and you've already done a
> bootstrap.
>
> _______________________________________________
> 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/20140527/f91a33c6/attachment.html>