[PATCH] D14004: Use the new 'InvariantInfo' property to eliminate redundant loads with --gvn.

Nick Lewycky via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 9 13:24:03 PST 2015


On 8 November 2015 at 21:04, Larisse Voufo <lvoufo at gmail.com> wrote:

> lvoufo added a comment.
>
> In http://reviews.llvm.org/D14004#284819, @nlewycky wrote:
>
> > This adds a call to "processInvariantIntrinsic" in GVN.cpp, but there is
> no such function and it isn't defined in this patch?
>
>
> processInvariantIntrinsic is defined in http://reviews.llvm.org/D14003,
> which this patch depends on. Does this help?
>

Aha! It does, thanks. I was treating them as all independent without
considering dependencies. Sorry!


>
>
> http://reviews.llvm.org/D14004
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151109/6fa8532b/attachment.html>


More information about the llvm-commits mailing list