[llvm-commits] [llvm] r165107 - /llvm/trunk/lib/Transforms/Instrumentation/ThreadSanitizer.cpp

[NAKAMURA Takumi]

Did you test?

FAIL: LLVM :: Instrumentation/ThreadSanitizer/atomic.ll (3678 of 6591)
******************** TEST 'LLVM ::
Instrumentation/ThreadSanitizer/atomic.ll' FAILED ********************
Script:
--
/home/chapuni/BUILD/cmake-static/Release/bin/./opt <
/home/chapuni/llvm-project/llvm/test/Instrumentation/ThreadSanitizer/atomic.ll
-tsan -S | /home/chapuni/BUILD/cmake-static/Release/bin/./FileCheck
/home/chapuni/llvm-project/llvm/test/Instrumentation/ThreadSanitizer/atomic.ll
--
Exit Code: 1
Command Output (stderr):
--
/home/chapuni/llvm-project/llvm/test/Instrumentation/ThreadSanitizer/atomic.ll:11:10:
error: expected string not found in input
; CHECK: call i8 @__tsan_atomic8_load(i8* %a, i32 1)
         ^
<stdin>:6:34: note: scanning from here
define i8 @atomic8_load_unordered(i8* %a) nounwind uwtable {
                                 ^
<stdin>:10:7: note: possible intended match here
 %1 = call i8 @__tsan_atomic8_load(i8* %a, i32 100501)
      ^
--


2012/10/3 Dmitry Vyukov <dvyukov at google.com>:
> Author: dvyukov
> Date: Wed Oct  3 08:00:57 2012
> New Revision: 165107
>
> URL: http://llvm.org/viewvc/llvm-project?rev=165107&view=rev
> Log:
> tsan: prepare for migration to new memory_order enum values (ABI compatible)
>
> Modified:
>     llvm/trunk/lib/Transforms/Instrumentation/ThreadSanitizer.cpp
>
> Modified: llvm/trunk/lib/Transforms/Instrumentation/ThreadSanitizer.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Instrumentation/ThreadSanitizer.cpp?rev=165107&r1=165106&r2=165107&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Transforms/Instrumentation/ThreadSanitizer.cpp (original)
> +++ llvm/trunk/lib/Transforms/Instrumentation/ThreadSanitizer.cpp Wed Oct  3 08:00:57 2012
> @@ -350,7 +350,8 @@
>      case AcquireRelease:         v = 1 << 4; break;
>      case SequentiallyConsistent: v = 1 << 5; break;
>    }
> -  return IRB->getInt32(v);
> +  // +100500 is temporal to migrate to new enum values.
> +  return IRB->getInt32(v + 100500);
>  }
>
>  bool ThreadSanitizer::instrumentAtomic(Instruction *I) {
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits