================
@@ -119,25 +119,15 @@ body: |
// which would return false in this case.
//
// There aren't any non-debug instruction between the beginning of bb1 and
- // Dbg1 (exclusive). However, the call to reset takes the end of the MBB as
- // the limit, so it pushes the beginning of the block up to %2's def and
- // considers the reset successful.
- EXPECT_TRUE(RPTracker.reset(*MBB1.begin(), &MBB1LiveIns));
- EXPECT_TRUE(RPTrackerNoLiveIns.reset(*MBB1.begin(), nullptr));
- // advance then unnecessarily processes instructions in order until the end
- // of the block, even though it is already past Dbg1. It still returns
false
- // because it is stopped by the end of block delimiter, not the end
- // iterator.
- EXPECT_FALSE(RPTracker.advance(Dbg1));
----------------
rovka wrote:
I think you should still test advance (just change the comment).
https://github.com/llvm/llvm-project/pull/196099
_______________________________________________
llvm-branch-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
