diff options
| author | Philip Reames <listmail@philipreames.com> | 2015-02-28 01:52:09 +0000 |
|---|---|---|
| committer | Philip Reames <listmail@philipreames.com> | 2015-02-28 01:52:09 +0000 |
| commit | af690c9cd32c482574c0999c795b1693065bbc53 (patch) | |
| tree | 1efab370324e2668bbb0ed641f630efa290a9483 /test | |
| parent | 82eef109be0bb7a0f323557290907a1a5915f326 (diff) | |
[RewriteStatepointsForGC] Fix another order of iteration bug
It turns out the naming of inserted phis and selects is sensative to the order in which two sets are iterated. We need to nail this down to avoid non-deterministic output and possible test failures.
The modified test is the one I first noticed something odd in. The change is making it more strict to report the error. With the test change, but without the code change, the test fails roughly 1 in 5. With the code change, I've run ~30 runs without error.
Long term, the right fix here is to adjust the naming scheme. I'm checking in this hack to avoid any possible non-determinism in the tests over the weekend. HJust because I only noticed one case doesn't mean it's actually the only case. I hope to get to the right change Monday.
std->llvm data structure changes bugfix change #3
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@230835 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'test')
| -rw-r--r-- | test/Transforms/RewriteStatepointsForGC/base-pointers.ll | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/Transforms/RewriteStatepointsForGC/base-pointers.ll b/test/Transforms/RewriteStatepointsForGC/base-pointers.ll index e7276821e1e..c5035eb3b1b 100644 --- a/test/Transforms/RewriteStatepointsForGC/base-pointers.ll +++ b/test/Transforms/RewriteStatepointsForGC/base-pointers.ll @@ -76,7 +76,7 @@ loop: ; preds = %loop, %entry ; CHECK-LABEL: loop ; CHECK: %base_phi = phi i64 addrspace(1)* ; CHECK-DAG: [ %base_obj, %entry ] -; CHECK-DAG: [ %base_select +; CHECK-DAG: [ %base_select.relocated, %loop ] ; CHECK-NOT: base_phi2 ; CHECK: next = select ; CHECK: base_select |