[PATCH] D14009: Handle info about the processing of invariant intrinsics...

Larisse Voufo via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 9 14:14:03 PST 2015


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

> On 8 November 2015 at 21:08, Larisse Voufo <lvoufo at gmail.com> wrote:
>
>> lvoufo added a comment.
>>
>> In http://reviews.llvm.org/D14009#284833, @nlewycky wrote:
>>
>> > This is overlapping with http://reviews.llvm.org/D14003?
>>
>>
>> Yes. It can be hard to keep track of the dependencies in this
>> framework... There is a comment in http://reviews.llvm.org/D14003 that
>> states that http://reviews.llvm.org/D14003 was split into
>> http://reviews.llvm.org/D14008 and http://reviews.llvm.org/D14009. So,
>> http://reviews.llvm.org/D14003 is really just a merge of both
>> http://reviews.llvm.org/D14008 and http://reviews.llvm.org/D14009.
>>
>
> Gotcha. I'll abandon this and review those instead.
>

No problem. I appreciate the comments. I'm reviewing the
AssumptionCache-like option... According to Philip, querying through  dom
tree info is another good option... I will push in revised patches once
I've made up my mind.


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


More information about the llvm-commits mailing list