[PATCH] D123720: [VPlan] Replace use of needsVectorIV with VPlan user check.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 8 10:54:11 PDT 2022


fhahn added a comment.

In D123720#3501663 <https://reviews.llvm.org/D123720#3501663>, @fhahn wrote:

> In D123720#3501646 <https://reviews.llvm.org/D123720#3501646>, @fhahn wrote:
>
>> In D123720#3501057 <https://reviews.llvm.org/D123720#3501057>, @alexfh wrote:
>>
>>> Do you see how to fix this quickly? Otherwise, please revert the commit while you're working on a fix. Thanks!
>>
>> Thanks for the test case! This highlights other parts that need to be cleaned up before landing this again. I've reverted the patch for now.

I recommitted the change now that all pending cleanups have landed.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D123720/new/

https://reviews.llvm.org/D123720



More information about the llvm-commits mailing list