[PATCH] D81369: [Alignment][NFC] Migrate AtomicExpandPass to Align

JF Bastien via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 9 09:19:35 PDT 2020


jfb added a comment.

Would you update anything in the langref w.r.t. these two instructions? The RMW langref doesn't say anything, and the cmpxchg part says:

> The pointer passed into cmpxchg must have alignment greater than or equal to the size in memory of the operand.

You're preserving the natural alignment requirement in the code, but langref seems slightly imprecise.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81369





More information about the llvm-commits mailing list