[PATCH] D136758: [ARM] Use DefaultAttrsIntrinsics

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 26 05:05:14 PDT 2022


nikic created this revision.
nikic added a reviewer: dmgreen.
Herald added a subscriber: kristof.beyls.
Herald added a project: All.
nikic requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

Use DefaultAttrsIntrinsics for most ARM intrinsics. This adds the WillReturn, NoSync, NoFree and NoCallback attributes and is needed to avoid regressions in the future.

I've switched to DefaultAttrIntrinsics for everything doing arithmetic and load/store. I've left some TODOs in cases where all DefaultsAttrs are not correct (e.g. ldrex etc are clearly not nosync) or it wasn't entirely obvious to me (e.g. stuff interacting with a coprocessor).


https://reviews.llvm.org/D136758

Files:
  llvm/include/llvm/IR/IntrinsicsARM.td
  llvm/test/CodeGen/ARM/vector-DAGCombine.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D136758.470786.patch
Type: text/x-patch
Size: 67872 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20221026/4fe7e87e/attachment.bin>


More information about the llvm-commits mailing list