[PATCH] D117795: [AArch64] Add some missing strict FP vector lowering
John Brawn via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Feb 1 09:45:21 PST 2022
john.brawn added inline comments.
================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:1484
+ // of the vector comparison instructions.
+ setOperationAction(ISD::STRICT_FSETCCS, VT, Expand);
+ // FIXME: We could potentially make use of the vector comparison instructions
----------------
dmgreen wrote:
> Can you split this into a separate patch? I know I sound like a broken record, but it doesn't seem to be related to the converts below.
>
> Also pre-committing as much of the test that works as possible would cut it down from this patch quite a bit.
Instead of a separate patch just for these two, it would probably make more sense to move them into D114946 with the rest of the setOperationAction lines.
On the test, without the changes in this patch it hits an assertion failure so as a separate commit before this it wouldn't be able to test anything.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D117795/new/
https://reviews.llvm.org/D117795
More information about the cfe-commits
mailing list