<div dir="ltr">LGTM.<div>Please just run the full make check and make sure you've caught all the places :)</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 21, 2016 at 12:17 PM, Mandeep Singh Grang <span dir="ltr"><<a href="mailto:mgrang@codeaurora.org" target="_blank">mgrang@codeaurora.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">mgrang created this revision.<br>
mgrang added reviewers: dberlin, david2050.<br>
mgrang added a subscriber: llvm-commits.<br>
mgrang set the repository for this revision to rL LLVM.<br>
<br>
<a href="https://reviews.llvm.org/D26704" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D26704</a> fixed the non-determinism in codegen by sorting basic blocks before<br>
iteration so as to have a defined iteration order. As a result we need to fix<br>
the names (numbers) of the temporaries in the following unit tests:<br>
<br>
  test/Transforms/Util/<wbr>MemorySSA/multi-edges.ll<br>
  test/Transforms/Util/<wbr>MemorySSA/multiple-backedges-<wbr>hal.ll<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D26926" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D26926</a><br>
<br>
Files:<br>
  test/Transforms/Util/<wbr>MemorySSA/multi-edges.ll<br>
  test/Transforms/Util/<wbr>MemorySSA/multiple-backedges-<wbr>hal.ll<br>
<br>
<br>
Index: test/Transforms/Util/<wbr>MemorySSA/multiple-backedges-<wbr>hal.ll<br>
==============================<wbr>==============================<wbr>=======<br>
--- test/Transforms/Util/<wbr>MemorySSA/multiple-backedges-<wbr>hal.ll<br>
+++ test/Transforms/Util/<wbr>MemorySSA/multiple-backedges-<wbr>hal.ll<br>
@@ -42,22 +42,22 @@<br>
   br label %OuterLoop<br>
<br>
 OuterLoop:<br>
-; CHECK: 5 = MemoryPhi({Entry,1},{<wbr>InnerLoop.Tail,3})<br>
+; CHECK: 4 = MemoryPhi({Entry,1},{<wbr>InnerLoop.Tail,3})<br>
 ; CHECK-NEXT: %Val.Outer =<br>
   %Val.Outer = call i8 @getValue()<br>
-; CHECK: 2 = MemoryDef(5)<br>
+; CHECK: 2 = MemoryDef(4)<br>
 ; CHECK-NEXT: store i8 %Val.Outer<br>
   store i8 %Val.Outer, i8* %Arg<br>
   call void @doThingWithoutReading()<br>
   br label %InnerLoop<br>
<br>
 InnerLoop:<br>
-; CHECK: 4 = MemoryPhi({OuterLoop,2},{<wbr>InnerLoop,3})<br>
-; CHECK-NEXT: ; MemoryUse(4)<br>
+; CHECK: 5 = MemoryPhi({OuterLoop,2},{<wbr>InnerLoop,3})<br>
+; CHECK-NEXT: ; MemoryUse(5)<br>
 ; CHECK-NEXT: %StartingAccess = load<br>
   %StartingAccess = load i8, i8* %Arg, align 4<br>
   %Val.Inner = call i8 @getValue()<br>
-; CHECK: 3 = MemoryDef(4)<br>
+; CHECK: 3 = MemoryDef(5)<br>
 ; CHECK-NEXT: store i8 %Val.Inner<br>
   store i8 %Val.Inner, i8* %Arg<br>
   call void @doThingWithoutReading()<br>
Index: test/Transforms/Util/<wbr>MemorySSA/multi-edges.ll<br>
==============================<wbr>==============================<wbr>=======<br>
--- test/Transforms/Util/<wbr>MemorySSA/multi-edges.ll<br>
+++ test/Transforms/Util/<wbr>MemorySSA/multi-edges.ll<br>
@@ -13,15 +13,15 @@<br>
   br i1 %a, label %Loop.Body, label %Loop.End<br>
<br>
 Loop.Body:<br>
-; CHECK: 4 = MemoryPhi({entry,1},{Loop.End,<wbr>3})<br>
-; CHECK-NEXT: 2 = MemoryDef(4)<br>
+; CHECK: 3 = MemoryPhi({entry,1},{Loop.End,<wbr>4})<br>
+; CHECK-NEXT: 2 = MemoryDef(3)<br>
 ; CHECK-NEXT: store i32 5<br>
   store i32 5, i32* %0, align 4<br>
   br i1 %a, label %Loop.End, label %Loop.End ; WhyDoWeEvenHaveThatLever.gif<br>
<br>
 Loop.End:<br>
-; CHECK: 3 = MemoryPhi({entry,1},{Loop.<wbr>Body,2},{Loop.Body,2})<br>
-; CHECK-NEXT: MemoryUse(3)<br>
+; CHECK: 4 = MemoryPhi({entry,1},{Loop.<wbr>Body,2},{Loop.Body,2})<br>
+; CHECK-NEXT: MemoryUse(4)<br>
 ; CHECK-NEXT: %1 = load<br>
   %1 = load i32, i32* %0, align 4<br>
   %2 = icmp eq i32 5, %1<br>
<br>
<br>
</blockquote></div><br></div>