[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