[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