[PATCH] D99545: [RISCV] Don't set the SplatOperand flag on intrinsics that take a shift amount.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 29 15:32:53 PDT 2021


craig.topper created this revision.
craig.topper added reviewers: khchen, HsiangKai, evandro, arcbbb.
Herald added subscribers: StephenFan, vkmr, frasercrmck, luismarques, apazos, sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei, PkmX, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, jrtc27, shiva0217, kito-cheng, niosHD, sabuasal, simoncook, johnrusso, rbar, asb.
craig.topper requested review of this revision.
Herald added a subscriber: MaskRay.
Herald added a project: LLVM.

The shift amount should always be a vector or an XLen scalar.
The SplatOperand flag is used to indicate we need to legalize
non-XLen scalars including special handling for i64 on RV32.
This will prevent us from silently adjusting these operands if
the intrinsics are misused.

I'll probably adjust the name of the SplatOperand flag slightly
in a follow up patch.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D99545

Files:
  llvm/include/llvm/IR/IntrinsicsRISCV.td

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D99545.333998.patch
Type: text/x-patch
Size: 8015 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210329/0b6236a0/attachment.bin>


More information about the llvm-commits mailing list