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

[Daniel Berlin via llvm-commits]

Larisse, for starters, you should make either a public version of that
doc or share that one (I don't know if it's possible to share it
outside our domain)


On Fri, Oct 9, 2015 at 2:48 PM, Philip Reames <listmail at philipreames.com> 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
>
>
> On 10/09/2015 02:21 PM, Larisse Voufo via llvm-commits wrote:
>
> lvoufo updated this revision to Diff 36992.
> lvoufo added a comment.
>
> Clang-formet'ed.
>
>
> http://reviews.llvm.org/D13606
>
> Files:
>   include/llvm/IR/GlobalVariable.h
>   include/llvm/IR/Instructions.h
>   lib/Analysis/BasicAliasAnalysis.cpp
>   lib/Analysis/Loads.cpp
>   lib/Analysis/MemoryDependenceAnalysis.cpp
>   lib/IR/Globals.cpp
>   lib/IR/Instructions.cpp
>   lib/Transforms/IPO/GlobalOpt.cpp
>   lib/Transforms/IPO/Inliner.cpp
>   lib/Transforms/InstCombine/InstCombineCalls.cpp
>   lib/Transforms/InstCombine/InstCombineInternal.h
>   lib/Transforms/InstCombine/InstructionCombining.cpp
>   lib/Transforms/Scalar/GVN.cpp
>   test/Transforms/LoadElim/global-local-vars.ll
>   test/Transforms/LoadElim/global-vars.ll
>   test/Transforms/LoadElim/local-vars.ll
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
>