[PATCH] D17833: [tsan] Add support for pointer typed atomic stores, loads, and cmpxchg

Dmitry Vyukov via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 3 23:44:23 PST 2016


dvyukov added inline comments.

================
Comment at: test/Instrumentation/ThreadSanitizer/atomic.ll:1196
@@ +1195,3 @@
+; CHECK: bitcast i8** %{{.+}} to i64*
+; CHECK-NEXT: call i64 @__tsan_atomic64_load(i64* %{{.+}}, i32 5), !dbg
+; CHECK-NEXT: inttoptr i64 %{{.+}} to i8*
----------------
Are you sure that we don't run these tests on 32-bit platforms? It will fail on 32-bit platform, right?


http://reviews.llvm.org/D17833





More information about the llvm-commits mailing list