[PATCH] D26704: [MemorySSA] Fix for non-determinism in codegen

Grang, Mandeep Singh via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 15 17:05:14 PST 2016


@dberlin With this new patch the IR seems to be the same each time. 
However the test cases need to be updated now to check for the correct 
temporaries.

For eg:

llvm-lit  -v test/Transforms/Util/MemorySSA/cyclicphi.ll

test/Transforms/Util/MemorySSA/cyclicphi.ll:14:10: error: expected 
string not found in input
; CHECK: 3 = MemoryPhi({%0,liveOnEntry},{bb77,2})
          ^
<stdin>:9:3: note: possible intended match here
; 2 = MemoryPhi({%0,liveOnEntry},{bb77,3})


On 11/15/2016 3:43 PM, Daniel Berlin wrote:
> dberlin added a comment.
>
> Here's an alternative diff that works and does not require slowing down IDFCalculator (when not all things care about the order of the end result):
> https://reviews.llvm.org/differential/diff/78091/
>
>
> https://reviews.llvm.org/D26704
>
>
>



More information about the llvm-commits mailing list