[PATCH] D97510: [AArch64][Docs] Release notes 12.x on outline atomics

Tim Northover via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 26 05:16:29 PST 2021


t.p.northover accepted this revision.
t.p.northover added a comment.
This revision is now accepted and ready to land.

I think this is fine.



================
Comment at: clang/docs/ReleaseNotes.rst:82
+  if compilation target supports LSE. Atomic instructions are used directly in
+  that case. The options behaviour mirrors GCC, the helpers are implemented
+  both in compiler-rt and libgcc.
----------------
Feel free to ignore since they're so minor, but if you're looking for more typos to fix:

  * The comma after "do not apply" is unnecessary.
  * "the compilation target" would be a bit more natural.
  * Missing apostrophe in "option's behaviour".

No need to reupload to Phab as far as I'm concerned either way.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97510



More information about the cfe-commits mailing list