[PATCH] D14004: Use the new 'InvariantInfo' property to eliminate redundant loads with --gvn.

[Larisse Voufo via llvm-commits]

On Mon, Nov 9, 2015 at 1:24 PM, Nick Lewycky <nlewycky at google.com> wrote:

> On 8 November 2015 at 21:04, Larisse Voufo <lvoufo at gmail.com> wrote:
>
>> lvoufo added a comment.
>>
>> In http://reviews.llvm.org/D14004#284819, @nlewycky wrote:
>>
>> > This adds a call to "processInvariantIntrinsic" in GVN.cpp, but there
>> is no such function and it isn't defined in this patch?
>>
>>
>> processInvariantIntrinsic is defined in http://reviews.llvm.org/D14003,
>> which this patch depends on. Does this help?
>>
>
> Aha! It does, thanks. I was treating them as all independent without
> considering dependencies. Sorry!
>

No need to apologize. It's easy things to turn messy with this system, as I
realize...


>
>>
>>
>> http://reviews.llvm.org/D14004
>>
>>
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151109/69695080/attachment.html>