[llvm] [VPlan] Move predication to VPlanTransform (NFC). (PR #128420)

via llvm-commits llvm-commits at lists.llvm.org
Tue May 20 07:17:46 PDT 2025


ayalz wrote:

> > May be worth reviewing the "native" VPlanPredicator logic introduced in https://reviews.llvm.org/D53349 and removed in https://reviews.llvm.org/D123017.
> 
> Might be good to as follow-up to potentially improve the predication implementation, once we completed the NFC move and completed the transition? Although the original l VPlanPredicator may need more work, as it was not enabled by default even in the native path and only tested via C++ unit tests.

Sure, just noting that this revives an older `VPlanPredicator.cpp` (along with its log?), which could offer some directions for improvements and/or extensions.

https://github.com/llvm/llvm-project/pull/128420


More information about the llvm-commits mailing list