<div dir="ltr">Oh man, the 'entry' block might be %0...  >_<<div><br></div><div>I didn't point this out in review because all the parameters retain their names in NDEBUG builds and nothing else is named.</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jun 18, 2014 at 2:48 PM, Saleem Abdulrasool <span dir="ltr"><<a href="mailto:compnerd@compnerd.org" target="_blank">compnerd@compnerd.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: compnerd<br>
Date: Wed Jun 18 16:48:44 2014<br>
New Revision: 211227<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=211227&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=211227&view=rev</a><br>
Log:<br>
tests: relax ms-intrinsics test<br>
<br>
Relax the tests to allow for differences between release and debug builds.  This<br>
should fix the buildbots.<br>
<br>
Thanks to Benjamin Kramer and Eric Christo for their invaluable tip that this<br>
was release build specific issue.<br>
<br>
Modified:<br>
    cfe/trunk/test/CodeGen/ms-intrinsics.c<br>
<br>
Modified: cfe/trunk/test/CodeGen/ms-intrinsics.c<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGen/ms-intrinsics.c?rev=211227&r1=211226&r2=211227&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGen/ms-intrinsics.c?rev=211227&r1=211226&r2=211227&view=diff</a><br>

==============================================================================<br>
--- cfe/trunk/test/CodeGen/ms-intrinsics.c (original)<br>
+++ cfe/trunk/test/CodeGen/ms-intrinsics.c Wed Jun 18 16:48:44 2014<br>
@@ -6,12 +6,11 @@ void *test_InterlockedExchangePointer(vo<br>
 }<br>
<br>
 // CHECK: define{{.*}}i8* @test_InterlockedExchangePointer(i8** %Target, i8* %Value){{.*}}{<br>
-// CHECK: entry:<br>
-// CHECK:   %0 = bitcast i8** %Target to i32*<br>
-// CHECK:   %1 = ptrtoint i8* %Value to i32<br>
-// CHECK:   %2 = atomicrmw xchg i32* %0, i32 %1 seq_cst<br>
-// CHECK:   %3 = inttoptr i32 %2 to i8*<br>
-// CHECK:   ret i8* %3<br>
+// CHECK:   %[[TARGET:[0-9]+]] = bitcast i8** %Target to i32*<br>
+// CHECK:   %[[VALUE:[0-9]+]] = ptrtoint i8* %Value to i32<br>
+// CHECK:   %[[EXCHANGE:[0-9]+]] = atomicrmw xchg i32* %[[TARGET]], i32 %[[VALUE]] seq_cst<br>
+// CHECK:   %[[RESULT:[0-9]+]] = inttoptr i32 %[[EXCHANGE]] to i8*<br>
+// CHECK:   ret i8* %[[RESULT]]<br>
 // CHECK: }<br>
<br>
 void *test_InterlockedCompareExchangePointer(void * volatile *Destination,<br>
@@ -20,14 +19,13 @@ void *test_InterlockedCompareExchangePoi<br>
 }<br>
<br>
 // CHECK: define{{.*}}i8* @test_InterlockedCompareExchangePointer(i8** %Destination, i8* %Exchange, i8* %Comparand){{.*}}{<br>
-// CHECK: entry:<br>
-// CHECK:   %0 = bitcast i8** %Destination to i32*<br>
-// CHECK:   %1 = ptrtoint i8* %Exchange to i32<br>
-// CHECK:   %2 = ptrtoint i8* %Comparand to i32<br>
-// CHECK:   %3 = cmpxchg volatile i32* %0, i32 %2, i32 %1 seq_cst seq_cst<br>
-// CHECK:   %4 = extractvalue { i32, i1 } %3, 0<br>
-// CHECK:   %5 = inttoptr i32 %4 to i8*<br>
-// CHECK:   ret i8* %5<br>
+// CHECK:   %[[DEST:[0-9]+]] = bitcast i8** %Destination to i32*<br>
+// CHECK:   %[[EXCHANGE:[0-9]+]] = ptrtoint i8* %Exchange to i32<br>
+// CHECK:   %[[COMPARAND:[0-9]+]] = ptrtoint i8* %Comparand to i32<br>
+// CHECK:   %[[XCHG:[0-9]+]] = cmpxchg volatile i32* %[[DEST:[0-9]+]], i32 %[[COMPARAND:[0-9]+]], i32 %[[EXCHANGE:[0-9]+]] seq_cst seq_cst<br>
+// CHECK:   %[[EXTRACT:[0-9]+]] = extractvalue { i32, i1 } %[[XCHG]], 0<br>
+// CHECK:   %[[RESULT:[0-9]+]] = inttoptr i32 %[[EXTRACT]] to i8*<br>
+// CHECK:   ret i8* %[[RESULT:[0-9]+]]<br>
 // CHECK: }<br>
<br>
 long test_InterlockedExchange(long *Target, long Value) {<br>
@@ -35,7 +33,6 @@ long test_InterlockedExchange(long *Targ<br>
 }<br>
<br>
 // CHECK: define{{.*}}i32 @test_InterlockedExchange(i32* %Target, i32 %Value){{.*}}{<br>
-// CHECK: entry:<br>
-// CHECK:   %0 = atomicrmw xchg i32* %Target, i32 %Value seq_cst<br>
-// CHECK:   ret i32 %0<br>
+// CHECK:   %[[EXCHANGE:[0-9]+]] = atomicrmw xchg i32* %Target, i32 %Value seq_cst<br>
+// CHECK:   ret i32 %[[EXCHANGE:[0-9]+]]<br>
 // CHECK: }<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</blockquote></div><br></div>