[PATCH] D38533: [CGP]Restrict complex select/phi case for optimizeMemoryInst

Serguei Katkov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 9 03:18:19 PDT 2017


skatkov added a comment.

In https://reviews.llvm.org/D38533#890238, @john.brawn wrote:

> There's no fundamental reason why a mixture of phi and select can't be handled, but this patch probably makes more sense in the context of later patches. I'll hold off reviewing it until then.


I tend to agree with you, right now I'm re-writing my main patch to support both Select and Phi at the same time. It seems it works. Most probably I will upload a first version of the patch tomorrow.


https://reviews.llvm.org/D38533





More information about the llvm-commits mailing list