[PATCH] D141189: [Mips] Set setMaxAtomicSizeInBitsSupported

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 8 10:12:32 PST 2023


efriedma added a comment.

> Wouldn't it be more correct to set atomic widths to 0 and generate __atomic_* calls?

You need some sort of operating-system level magic to make atomicrmw/cmpxchg work without ll/sc.  Disabling interrupts, or a restartable sequence, or something along those lines.  Most of the ways you'd  implement that end up being compatible with lowering atomic load/store to plain load/store operations.  Granted, other implementations are possible.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D141189/new/

https://reviews.llvm.org/D141189



More information about the llvm-commits mailing list