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

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 9 12:36:50 PDT 2022


fhahn added a comment.

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.

D124936 <https://reviews.llvm.org/D124936> is part of the required cleanup.


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