[PATCH] D99434: [TSAN] Honor failure memory orders in AtomicCAS

Bruno Cardoso Lopes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 6 22:38:44 PDT 2021


bruno updated this revision to Diff 343581.
bruno retitled this revision from "[TSAN] Honor acquire failure mode on AtomicCAS" to "[TSAN] Honor failure memory orders in AtomicCAS".
bruno edited the summary of this revision.
bruno added a comment.

Apply more reviewer suggested changes: first evaluate CAS, then decide what order to use.

Now that LLVM side of this is fixed, also cover all fmo's allowed, add more tests.


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

https://reviews.llvm.org/D99434

Files:
  compiler-rt/lib/tsan/rtl/tsan_interface_atomic.cpp
  compiler-rt/test/tsan/compare_exchange.cpp
  llvm/test/Instrumentation/ThreadSanitizer/atomic.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D99434.343581.patch
Type: text/x-patch
Size: 11992 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210507/7db7ca31/attachment.bin>


More information about the llvm-commits mailing list