[PATCH] D112028: [RISCV] Remove the HasSideEffects property from riscv_vector.td

Craig Topper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 18 12:31:03 PDT 2021


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

It was being used to control the nothrow attribute on the builtins. The
nothrow attribute is for C++ exceptions. Even if the vector builtins
have side effects in IR, that's different than the nothrow attribute.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D112028

Files:
  clang/include/clang/Basic/riscv_vector.td
  clang/utils/TableGen/RISCVVEmitter.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D112028.380496.patch
Type: text/x-patch
Size: 5792 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20211018/ceeed51f/attachment-0001.bin>


More information about the cfe-commits mailing list