<div dir="ltr">Okay. I've made an external copy available.</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 9, 2015 at 2:52 PM, Daniel Berlin <span dir="ltr"><<a href="mailto:dberlin@dberlin.org" target="_blank">dberlin@dberlin.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Larisse, for starters, you should make either a public version of that<br>
doc or share that one (I don't know if it's possible to share it<br>
outside our domain)<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
On Fri, Oct 9, 2015 at 2:48 PM, Philip Reames <<a href="mailto:listmail@philipreames.com">listmail@philipreames.com</a>> wrote:<br>
> This definitely needs to be split up into a series of smaller patches to be<br>
> reviewable.  I strongly suggest starting small, getting through one cycle of<br>
> review, then posting a couple more and recursing.  Reading through the diff,<br>
> I see a lot of points which will need discussion/clarification.<br>
><br>
> Also, a bit of background on what you're trying to accomplish is required.<br>
> A link to a private google doc is not sufficient.<br>
><br>
> Philip<br>
><br>
><br>
> On 10/09/2015 02:21 PM, Larisse Voufo via llvm-commits wrote:<br>
><br>
> lvoufo updated this revision to Diff 36992.<br>
> lvoufo added a comment.<br>
><br>
> Clang-formet'ed.<br>
><br>
><br>
> <a href="http://reviews.llvm.org/D13606" rel="noreferrer" target="_blank">http://reviews.llvm.org/D13606</a><br>
><br>
> Files:<br>
>   include/llvm/IR/GlobalVariable.h<br>
>   include/llvm/IR/Instructions.h<br>
>   lib/Analysis/BasicAliasAnalysis.cpp<br>
>   lib/Analysis/Loads.cpp<br>
>   lib/Analysis/MemoryDependenceAnalysis.cpp<br>
>   lib/IR/Globals.cpp<br>
>   lib/IR/Instructions.cpp<br>
>   lib/Transforms/IPO/GlobalOpt.cpp<br>
>   lib/Transforms/IPO/Inliner.cpp<br>
>   lib/Transforms/InstCombine/InstCombineCalls.cpp<br>
>   lib/Transforms/InstCombine/InstCombineInternal.h<br>
>   lib/Transforms/InstCombine/InstructionCombining.cpp<br>
>   lib/Transforms/Scalar/GVN.cpp<br>
>   test/Transforms/LoadElim/global-local-vars.ll<br>
>   test/Transforms/LoadElim/global-vars.ll<br>
>   test/Transforms/LoadElim/local-vars.ll<br>
><br>
><br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
><br>
><br>
</div></div></blockquote></div><br></div>