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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
