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

Philip Reames via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 9 14:48:35 PDT 2015


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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151009/e2e6377a/attachment.html>


More information about the llvm-commits mailing list