[PATCH] D13606: [Introduction] Redundant load reduction with invariant intrinsics

Larisse Voufo via llvm-commits llvm-commits at lists.llvm.org
Sun Nov 8 21:18:52 PST 2015


On Sun, Nov 8, 2015 at 3:34 PM, Nick Lewycky <nlewycky at google.com> wrote:

> On 22 October 2015 at 21:36, Larisse Voufo <lvoufo at gmail.com> wrote:
>
>> lvoufo marked an inline comment as done.
>> lvoufo added a comment.
>>
>> I have addressed all the feedback so far (Thanks, Nick!) and split the
>> updated version of this patch into 5 patches:
>> http://reviews.llvm.org/D14003, http://reviews.llvm.org/D14004,
>> http://reviews.llvm.org/D14005, http://reviews.llvm.org/D14006 and
>> http://reviews.llvm.org/D14007.
>> Is this clearer? Would it help to split it up further?
>>
>
> Is this review still live or do those 5 supercede this one?
>

Those 5 supercede this one.


>
>
>> Thanks,
>>
>> - Larisse.
>>
>>
>> http://reviews.llvm.org/D13606
>>
>>
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151108/94cd5454/attachment.html>


More information about the llvm-commits mailing list