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

Larisse Voufo via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 9 17:04:52 PDT 2015


lvoufo added a comment.

In http://reviews.llvm.org/D13606#263966, @reames wrote:

> This definitely needs to be split up into a series of smaller patches to 
>  be reviewable.  I strongly suggest starting small, getting through one 
>  cycle of review, then posting a couple more and recursing.  Reading 
>  through the diff, I see a lot of points which will need 
>  discussion/clarification.
>
> Also, a bit of background on what you're trying to accomplish is 
>  required.  A link to a private google doc is not sufficient.
>
> Philip


Hi Philip -- This *is* the initial patch. I can't think of another way to break it down more effectively. 
The design doc is not meant to be private. If you are unable to access, I believe you should be able to  request access, which I will promptly grant (?).
Will be giving this a bit more thought. Thanks!


http://reviews.llvm.org/D13606





More information about the llvm-commits mailing list