[llvm] [VPlan][NFC] Decide consecutive mem-op predication from VPlan masks. (PR #215537)
Madhur Amilkanthwar via llvm-commits
llvm-commits at lists.llvm.org
Wed Aug 12 06:07:59 PDT 2026
madhur13490 wrote:
> I think this is unfortunately incorrect, and there is more work to do on the predicator before we can get rid of the legacy getPredicatedInst :(
Thanks. Can you please elaborate what are all things missing here? I haven't looked deep into the predicator.
For this patch, do you think something like this would be useful when we try to drop the legacy path?
```
assert(VPI->isMasked() == RecipeBuilder.isPredicatedInst(I) &&
"VPlan mask disagrees with legacy predication query");
if (RecipeBuilder.isPredicatedInst(I) || !IsLoad || ...
```
https://github.com/llvm/llvm-project/pull/215537
More information about the llvm-commits
mailing list