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

[Dmitry Vyukov]

No, I don't. I yet need to figure out how to run tests.


On Wed, Oct 3, 2012 at 5:11 PM, NAKAMURA Takumi <geek4civic at gmail.com>wrote:

> 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
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20121003/359c7ecf/attachment.html>