[PATCH] D75452: [ARM][MVE] Validate tail predication values
Sam Parker via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 10 02:19:52 PDT 2020
samparker marked an inline comment as done.
samparker added a comment.
> do we have test where lanes are swapped (if that makes sense)?
I don't think there's one for this pass... but that was the reason for having a unit test for validForTailPredication, so we don't have to write a whole loop to test an instruction.
================
Comment at: llvm/lib/Target/ARM/ARMLowOverheadLoops.cpp:582
+ // Only evaluate instructions which produce a single value.
+ if (MI.getNumDefs() != 1 || !MI.defs().begin()->isReg()) {
+ UnknownFalseLanes.insert(&MI);
----------------
SjoerdMeijer wrote:
> Is there a test for this?
The only instructions that I can think of would be indexed load/stores, but they'll be caught by isVectorPredicated. If/When we can enable reductions, we could have an instruction producing two scalar values and then this logic will need to be reorganised a bit.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75452/new/
https://reviews.llvm.org/D75452
More information about the llvm-commits
mailing list