<div dir="ltr">It would seem wise to change this test to use a more obvious way to disable the CHECK lines, such as by replacing CHECK with DISABLED instead of adding a space in the middle of the -NEXT, so that another enthusiastic janitor is less likely to remove the spaces again.</div><br><div class="gmail_quote"><div dir="ltr">On Thu, 5 Jul 2018 at 04:30, Krasimir Georgiev via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: krasimir<br>
Date: Thu Jul  5 04:30:15 2018<br>
New Revision: 336336<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=336336&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=336336&view=rev</a><br>
Log:<br>
Partially revert r336268 in address-offsets.ll<br>
<br>
Summary: There the typos are intentional, explicitly introduced to disable these cases in r280285.<br>
<br>
Reviewers: bkramer<br>
<br>
Reviewed By: bkramer<br>
<br>
Subscribers: dschuff, sbc100, jgravelle-google, aheejin, llvm-commits<br>
<br>
Differential Revision: <a href="https://reviews.llvm.org/D48962" rel="noreferrer" target="_blank">https://reviews.llvm.org/D48962</a><br>
<br>
Modified:<br>
    llvm/trunk/test/CodeGen/WebAssembly/address-offsets.ll<br>
<br>
Modified: llvm/trunk/test/CodeGen/WebAssembly/address-offsets.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/WebAssembly/address-offsets.ll?rev=336336&r1=336335&r2=336336&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/WebAssembly/address-offsets.ll?rev=336336&r1=336335&r2=336336&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/CodeGen/WebAssembly/address-offsets.ll (original)<br>
+++ llvm/trunk/test/CodeGen/WebAssembly/address-offsets.ll Thu Jul  5 04:30:15 2018<br>
@@ -36,9 +36,9 @@ define i32 @load_test0_noinbounds() {<br>
 ; CHECK-NEXT: param     i32{{$}}<br>
 ; CHECK-NEXT: result    i32{{$}}<br>
 ; CHECK-NEXT: i32.const $push0=, 2{{$}}<br>
-; CHECK-NEXT: i32.shl   $push1=, $0, $pop0{{$}}<br>
-; CHECK-NEXT: i32.load  $push2=, g+40($pop1){{$}}<br>
-; CHECK-NEXT: return    $pop2{{$}}<br>
+; CHECK-NEX T: i32.shl   $push1=, $0, $pop0{{$}}<br>
+; CHECK-NEX T: i32.load  $push2=, g+40($pop1){{$}}<br>
+; CHECK-NEX T: return    $pop2{{$}}<br>
 define i32 @load_test1(i32 %n) {<br>
   %add = add nsw i32 %n, 10<br>
   %arrayidx = getelementptr inbounds [0 x i32], [0 x i32]* @g, i32 0, i32 %add<br>
@@ -50,9 +50,9 @@ define i32 @load_test1(i32 %n) {<br>
 ; CHECK-NEXT: param     i32{{$}}<br>
 ; CHECK-NEXT: result    i32{{$}}<br>
 ; CHECK-NEXT: i32.const $push0=, 2{{$}}<br>
-; CHECK-NEXT: i32.shl   $push1=, $0, $pop0{{$}}<br>
-; CHECK-NEXT: i32.load  $push2=, g+40($pop1){{$}}<br>
-; CHECK-NEXT: return    $pop2{{$}}<br>
+; CHECK-NEX T: i32.shl   $push1=, $0, $pop0{{$}}<br>
+; CHECK-NEX T: i32.load  $push2=, g+40($pop1){{$}}<br>
+; CHECK-NEX T: return    $pop2{{$}}<br>
 define i32 @load_test2(i32 %n) {<br>
   %add = add nsw i32 10, %n<br>
   %arrayidx = getelementptr inbounds [0 x i32], [0 x i32]* @g, i32 0, i32 %add<br>
@@ -64,9 +64,9 @@ define i32 @load_test2(i32 %n) {<br>
 ; CHECK-NEXT: param     i32{{$}}<br>
 ; CHECK-NEXT: result    i32{{$}}<br>
 ; CHECK-NEXT: i32.const $push0=, 2{{$}}<br>
-; CHECK-NEXT: i32.shl   $push1=, $0, $pop0{{$}}<br>
-; CHECK-NEXT: i32.load  $push2=, g+40($pop1){{$}}<br>
-; CHECK-NEXT: return    $pop2{{$}}<br>
+; CHECK-NEX T: i32.shl   $push1=, $0, $pop0{{$}}<br>
+; CHECK-NEX T: i32.load  $push2=, g+40($pop1){{$}}<br>
+; CHECK-NEX T: return    $pop2{{$}}<br>
 define i32 @load_test3(i32 %n) {<br>
   %add.ptr = getelementptr inbounds [0 x i32], [0 x i32]* @g, i32 0, i32 %n<br>
   %add.ptr1 = getelementptr inbounds i32, i32* %add.ptr, i32 10<br>
@@ -78,9 +78,9 @@ define i32 @load_test3(i32 %n) {<br>
 ; CHECK-NEXT: param     i32{{$}}<br>
 ; CHECK-NEXT: result    i32{{$}}<br>
 ; CHECK-NEXT: i32.const $push0=, 2{{$}}<br>
-; CHECK-NEXT: i32.shl   $push1=, $0, $pop0{{$}}<br>
-; CHECK-NEXT: i32.load  $push2=, g+40($pop1){{$}}<br>
-; CHECK-NEXT: return    $pop2{{$}}<br>
+; CHECK-NEX T: i32.shl   $push1=, $0, $pop0{{$}}<br>
+; CHECK-NEX T: i32.load  $push2=, g+40($pop1){{$}}<br>
+; CHECK-NEX T: return    $pop2{{$}}<br>
 define i32 @load_test4(i32 %n) {<br>
   %add.ptr = getelementptr inbounds i32, i32* getelementptr inbounds ([0 x i32], [0 x i32]* @g, i32 0, i32 10), i32 %n<br>
   %t = load i32, i32* %add.ptr, align 4<br>
@@ -91,9 +91,9 @@ define i32 @load_test4(i32 %n) {<br>
 ; CHECK-NEXT: param     i32{{$}}<br>
 ; CHECK-NEXT: result    i32{{$}}<br>
 ; CHECK-NEXT: i32.const $push0=, 2{{$}}<br>
-; CHECK-NEXT: i32.shl   $push1=, $0, $pop0{{$}}<br>
-; CHECK-NEXT: i32.load  $push2=, g+40($pop1){{$}}<br>
-; CHECK-NEXT: return    $pop2{{$}}<br>
+; CHECK-NEX T: i32.shl   $push1=, $0, $pop0{{$}}<br>
+; CHECK-NEX T: i32.load  $push2=, g+40($pop1){{$}}<br>
+; CHECK-NEX T: return    $pop2{{$}}<br>
 define i32 @load_test5(i32 %n) {<br>
   %add.ptr = getelementptr inbounds i32, i32* getelementptr inbounds ([0 x i32], [0 x i32]* @g, i32 0, i32 10), i32 %n<br>
   %t = load i32, i32* %add.ptr, align 4<br>
@@ -104,9 +104,9 @@ define i32 @load_test5(i32 %n) {<br>
 ; CHECK-NEXT: param     i32{{$}}<br>
 ; CHECK-NEXT: result    i32{{$}}<br>
 ; CHECK-NEXT: i32.const $push0=, 2{{$}}<br>
-; CHECK-NEXT: i32.shl   $push1=, $0, $pop0{{$}}<br>
-; CHECK-NEXT: i32.load  $push2=, g+40($pop1){{$}}<br>
-; CHECK-NEXT: return    $pop2{{$}}<br>
+; CHECK-NEX T: i32.shl   $push1=, $0, $pop0{{$}}<br>
+; CHECK-NEX T: i32.load  $push2=, g+40($pop1){{$}}<br>
+; CHECK-NEX T: return    $pop2{{$}}<br>
 define i32 @load_test6(i32 %n) {<br>
   %add = add nsw i32 %n, 10<br>
   %add.ptr = getelementptr inbounds [0 x i32], [0 x i32]* @g, i32 0, i32 %add<br>
@@ -118,9 +118,9 @@ define i32 @load_test6(i32 %n) {<br>
 ; CHECK-NEXT: param     i32{{$}}<br>
 ; CHECK-NEXT: result    i32{{$}}<br>
 ; CHECK-NEXT: i32.const $push0=, 2{{$}}<br>
-; CHECK-NEXT: i32.shl   $push1=, $0, $pop0{{$}}<br>
-; CHECK-NEXT: i32.load  $push2=, g+40($pop1){{$}}<br>
-; CHECK-NEXT: return    $pop2{{$}}<br>
+; CHECK-NEX T: i32.shl   $push1=, $0, $pop0{{$}}<br>
+; CHECK-NEX T: i32.load  $push2=, g+40($pop1){{$}}<br>
+; CHECK-NEX T: return    $pop2{{$}}<br>
 define i32 @load_test7(i32 %n) {<br>
   %add.ptr = getelementptr inbounds [0 x i32], [0 x i32]* @g, i32 0, i32 %n<br>
   %add.ptr1 = getelementptr inbounds i32, i32* %add.ptr, i32 10<br>
@@ -132,9 +132,9 @@ define i32 @load_test7(i32 %n) {<br>
 ; CHECK-NEXT: param     i32{{$}}<br>
 ; CHECK-NEXT: result    i32{{$}}<br>
 ; CHECK-NEXT: i32.const $push0=, 2{{$}}<br>
-; CHECK-NEXT: i32.shl   $push1=, $0, $pop0{{$}}<br>
-; CHECK-NEXT: i32.load  $push2=, g+40($pop1){{$}}<br>
-; CHECK-NEXT: return    $pop2{{$}}<br>
+; CHECK-NEX T: i32.shl   $push1=, $0, $pop0{{$}}<br>
+; CHECK-NEX T: i32.load  $push2=, g+40($pop1){{$}}<br>
+; CHECK-NEX T: return    $pop2{{$}}<br>
 define i32 @load_test8(i32 %n) {<br>
   %add = add nsw i32 10, %n<br>
   %add.ptr = getelementptr inbounds [0 x i32], [0 x i32]* @g, i32 0, i32 %add<br>
@@ -378,8 +378,8 @@ define void @store_test0_noinbounds(i32<br>
 ; CHECK-NEXT: param     i32, i32{{$}}<br>
 ; CHECK-NEXT: i32.const $push0=, 2{{$}}<br>
 ; CHECK-NEXT: i32.shl   $push1=, $0, $pop0{{$}}<br>
-; CHECK-NEXT: i32.store g+40($pop1), $1{{$}}<br>
-; CHECK-NEXT: return{{$}}<br>
+; CHECK-NEX T: i32.store g+40($pop1), $1{{$}}<br>
+; CHECK-NEX T: return{{$}}<br>
 define void @store_test1(i32 %n, i32 %i) {<br>
   %add = add nsw i32 %n, 10<br>
   %arrayidx = getelementptr inbounds [0 x i32], [0 x i32]* @g, i32 0, i32 %add<br>
@@ -391,8 +391,8 @@ define void @store_test1(i32 %n, i32 %i)<br>
 ; CHECK-NEXT: param     i32, i32{{$}}<br>
 ; CHECK-NEXT: i32.const $push0=, 2{{$}}<br>
 ; CHECK-NEXT: i32.shl   $push1=, $0, $pop0{{$}}<br>
-; CHECK-NEXT: i32.store g+40($pop1), $1{{$}}<br>
-; CHECK-NEXT: return{{$}}<br>
+; CHECK-NEX T: i32.store g+40($pop1), $1{{$}}<br>
+; CHECK-NEX T: return{{$}}<br>
 define void @store_test2(i32 %n, i32 %i) {<br>
   %add = add nsw i32 10, %n<br>
   %arrayidx = getelementptr inbounds [0 x i32], [0 x i32]* @g, i32 0, i32 %add<br>
@@ -404,8 +404,8 @@ define void @store_test2(i32 %n, i32 %i)<br>
 ; CHECK-NEXT: param     i32, i32{{$}}<br>
 ; CHECK-NEXT: i32.const $push0=, 2{{$}}<br>
 ; CHECK-NEXT: i32.shl   $push1=, $0, $pop0{{$}}<br>
-; CHECK-NEXT: i32.store g+40($pop1), $1{{$}}<br>
-; CHECK-NEXT: return{{$}}<br>
+; CHECK-NEX T: i32.store g+40($pop1), $1{{$}}<br>
+; CHECK-NEX T: return{{$}}<br>
 define void @store_test3(i32 %n, i32 %i) {<br>
   %add.ptr = getelementptr inbounds [0 x i32], [0 x i32]* @g, i32 0, i32 %n<br>
   %add.ptr1 = getelementptr inbounds i32, i32* %add.ptr, i32 10<br>
@@ -417,8 +417,8 @@ define void @store_test3(i32 %n, i32 %i)<br>
 ; CHECK-NEXT: param     i32, i32{{$}}<br>
 ; CHECK-NEXT: i32.const $push0=, 2{{$}}<br>
 ; CHECK-NEXT: i32.shl   $push1=, $0, $pop0{{$}}<br>
-; CHECK-NEXT: i32.store g+40($pop1), $1{{$}}<br>
-; CHECK-NEXT: return{{$}}<br>
+; CHECK-NEX T: i32.store g+40($pop1), $1{{$}}<br>
+; CHECK-NEX T: return{{$}}<br>
 define void @store_test4(i32 %n, i32 %i) {<br>
   %add.ptr = getelementptr inbounds i32, i32* getelementptr inbounds ([0 x i32], [0 x i32]* @g, i32 0, i32 10), i32 %n<br>
   store i32 %i, i32* %add.ptr, align 4<br>
@@ -429,8 +429,8 @@ define void @store_test4(i32 %n, i32 %i)<br>
 ; CHECK-NEXT: param     i32, i32{{$}}<br>
 ; CHECK-NEXT: i32.const $push0=, 2{{$}}<br>
 ; CHECK-NEXT: i32.shl   $push1=, $0, $pop0{{$}}<br>
-; CHECK-NEXT: i32.store g+40($pop1), $1{{$}}<br>
-; CHECK-NEXT: return{{$}}<br>
+; CHECK-NEX T: i32.store g+40($pop1), $1{{$}}<br>
+; CHECK-NEX T: return{{$}}<br>
 define void @store_test5(i32 %n, i32 %i) {<br>
   %add.ptr = getelementptr inbounds i32, i32* getelementptr inbounds ([0 x i32], [0 x i32]* @g, i32 0, i32 10), i32 %n<br>
   store i32 %i, i32* %add.ptr, align 4<br>
@@ -441,8 +441,8 @@ define void @store_test5(i32 %n, i32 %i)<br>
 ; CHECK-NEXT: param     i32, i32{{$}}<br>
 ; CHECK-NEXT: i32.const $push0=, 2{{$}}<br>
 ; CHECK-NEXT: i32.shl   $push1=, $0, $pop0{{$}}<br>
-; CHECK-NEXT: i32.store g+40($pop1), $1{{$}}<br>
-; CHECK-NEXT: return{{$}}<br>
+; CHECK-NEX T: i32.store g+40($pop1), $1{{$}}<br>
+; CHECK-NEX T: return{{$}}<br>
 define void @store_test6(i32 %n, i32 %i) {<br>
   %add = add nsw i32 %n, 10<br>
   %add.ptr = getelementptr inbounds [0 x i32], [0 x i32]* @g, i32 0, i32 %add<br>
@@ -454,8 +454,8 @@ define void @store_test6(i32 %n, i32 %i)<br>
 ; CHECK-NEXT: param     i32, i32{{$}}<br>
 ; CHECK-NEXT: i32.const $push0=, 2{{$}}<br>
 ; CHECK-NEXT: i32.shl   $push1=, $0, $pop0{{$}}<br>
-; CHECK-NEXT: i32.store g+40($pop1), $1{{$}}<br>
-; CHECK-NEXT: return{{$}}<br>
+; CHECK-NEX T: i32.store g+40($pop1), $1{{$}}<br>
+; CHECK-NEX T: return{{$}}<br>
 define void @store_test7(i32 %n, i32 %i) {<br>
   %add.ptr = getelementptr inbounds [0 x i32], [0 x i32]* @g, i32 0, i32 %n<br>
   %add.ptr1 = getelementptr inbounds i32, i32* %add.ptr, i32 10<br>
@@ -467,8 +467,8 @@ define void @store_test7(i32 %n, i32 %i)<br>
 ; CHECK-NEXT: param     i32, i32{{$}}<br>
 ; CHECK-NEXT: i32.const $push0=, 2{{$}}<br>
 ; CHECK-NEXT: i32.shl   $push1=, $0, $pop0{{$}}<br>
-; CHECK-NEXT: i32.store g+40($pop1), $1{{$}}<br>
-; CHECK-NEXT: return{{$}}<br>
+; CHECK-NEX T: i32.store g+40($pop1), $1{{$}}<br>
+; CHECK-NEX T: return{{$}}<br>
 define void @store_test8(i32 %n, i32 %i) {<br>
   %add = add nsw i32 10, %n<br>
   %add.ptr = getelementptr inbounds [0 x i32], [0 x i32]* @g, i32 0, i32 %add<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>