[RewriteStatepointsForGC] Bugfix for change 246133
authorPhilip Reames <listmail@philipreames.com>
Wed, 2 Sep 2015 22:25:07 +0000 (22:25 +0000)
committerPhilip Reames <listmail@philipreames.com>
Wed, 2 Sep 2015 22:25:07 +0000 (22:25 +0000)
commit3f945c295e6add83a10058f9ec85d36bdf97cf96
tree89652008a1ba2defc804db907c928975c25ebfd8
parent774bfb2915f6c0d838a432e8ecf69fdf25956e1c
[RewriteStatepointsForGC] Bugfix for change 246133

Fix a bug in change 246133. I didn't handle the case where we had a cycle in the use graph and could add an instruction we were about to erase back on to the worklist. Oddly, I have not been able to write a small test case for this, even with the AssertingVH added. I have confirmed the basic theory for the fix on a large failing example, but all attempts to reduce that to something appropriate for a test case have failed.

Differential Revision: http://reviews.llvm.org/D12575

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@246718 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Transforms/Scalar/RewriteStatepointsForGC.cpp