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

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


Okay. I've made an external copy available.

On Fri, Oct 9, 2015 at 2:52 PM, Daniel Berlin <dberlin at dberlin.org> wrote:

> 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
> >
> >
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151009/3ee7c77c/attachment.html>


More information about the llvm-commits mailing list