[PATCH] D14271: [CVP] Fold return values if possible

Nick Lewycky via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 2 18:40:14 PST 2015


On 2 November 2015 at 18:13, Philip Reames <listmail at philipreames.com>
wrote:

> reames added inline comments.
>
> ================
> Comment at: lib/Transforms/Scalar/CorrelatedValuePropagation.cpp:339-340
> @@ +338,4 @@
> +  // at least the outer shims around such.
> +  auto *C = dyn_cast<CmpInst>(V);
> +  if (!C) return nullptr;
> +
> ----------------
> Do you have a particular pattern you'd want to handle?  My preference
> would be a) a follow on change and b) not doing this until I can get this
> pushed down into LVI.  (That's on my list, just need to clear some other
> LVI/CVP changes out first.)
>

No particular pattern, I don't mind waiting until after it gets sunk into
LVI. Thanks!
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151102/d17386a4/attachment.html>


More information about the llvm-commits mailing list