Add SVE2 mla indexed intrinsics:
- smlalb, smalalt, umlalb, umlalt, smlslb, smlslt, umlslb, umlslt.
Paths
| Differential D73576
[AArch64][SVE] Add SVE2 mla indexed intrinsics ClosedPublic Authored by dancgr on Jan 28 2020, 1:06 PM.
Details Summary Add SVE2 mla indexed intrinsics:
Diff Detail
Event Timelinedancgr retitled this revision from [AArch64][SVE] Add SVE2 mla indexed intrinsics. to [AArch64][SVE] Add SVE2 mla indexed intrinsics.Jan 28 2020, 1:17 PM This revision is now accepted and ready to land.Jan 28 2020, 1:28 PM Closed by commit rG1f85dfb2af1a: [AArch64][SVE] Add SVE2 mla indexed intrinsics. (authored by dancgr). · Explain WhyJan 28 2020, 2:15 PM This revision was automatically updated to reflect the committed changes. Comment Actions @dancgr I see you already committed the patch, but could you please still address my comments?
Comment Actions I will make the changes suggested by Sander in a following patch, joined with the saturating multiply-add long intirnsics.
Revision Contents
Diff 240978 llvm/include/llvm/IR/IntrinsicsAArch64.td
llvm/lib/Target/AArch64/AArch64SVEInstrInfo.td
llvm/lib/Target/AArch64/SVEInstrFormats.td
llvm/test/CodeGen/AArch64/sve2-mla-indexed.ll
|
For consistency with other intrinsics, these need to use an i32 value for the immediate. The reason is that the front-end will generate the calls to the intrinsics automatically and will use an i32 for the immediates. Having one exception to the rule means this is something we'll need to fix in Clang or definition of the intrinsic later.