[llvm] [AArch64] Don't replace dst of SWP instructions with (X|W)ZR (PR #102139)

Luke Geeson via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 6 23:21:06 PDT 2024


lukeg101 wrote:

thanks for fixing this @pratlucas !

the patch looks good to me, I note that this semantics applies to CAS/SWP/LD<op> and ST<OP> variants, so if CAS is used as the target of an atomic in the future in a similar fashion, the behaviour will arise again. Currently CAS is used in other op that requires the dst, and so it’s fine for now.

> Should this be backported to 19.x?

Yes I think so, it’s relatively harmless a fix but ensures translation of atomic RMW intrinsics, it shouldn’t affect other backends 

https://github.com/llvm/llvm-project/pull/102139


More information about the llvm-commits mailing list