[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