[PATCH] D97987: AArch64: Set MaxAtomicSizeInBitsSupported appropriately.

[James Y Knight via Phabricator via llvm-commits]

jyknight created this revision.
jyknight added a reviewer: t.p.northover.
Herald added subscribers: danielkiss, jfb, hiraditya, kristof.beyls.
jyknight requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

This will result in larger atomic operations getting expanded to
__atomic_* libcalls via AtomicExpandPass. This is a part of a change
to similarly clean up atomics handling on all targets.

AArch64 always supports 128-bit atomics, so this is nice and simple.

Additionally, adjust some comments, and remove partial code dealing
with larger-than-128bit atomics, as it's now unreachable.

The arm64-irtranslator.ll test was adjusted as it was using an i258
type as a hack to avoid IR atomic lowering, and test GlobalISel
behavior. Pass -mattr=+lse and use i32, instead, to accomplish that
goal in a more reasonable manner.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D97987

Files:
  llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
  llvm/test/CodeGen/AArch64/GlobalISel/arm64-irtranslator.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D97987.328323.patch
Type: text/x-patch
Size: 16474 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210304/9ef5e7b2/attachment.bin>