llvm.org GIT mirror llvm / ab0a33b
Rename skipDebugInfo -> skipDebugIntrinsics, NFC This addresses post-commit feedback about the name 'skipDebugInfo' being misleading. This name could be interpreted as meaning 'a function that skips instructions with debug locations'. The new name, 'skipDebugIntrinsics', makes it clear that this function only skips debug info intrinsics. Thanks to Adrian Prantl for pointing this out! git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@335667 91177308-0d34-0410-b5e6-96231b3b80d8 Vedant Kumar 1 year, 3 months ago
3 changed file(s) with 3 addition(s) and 3 deletion(s). Raw diff Collapse all Expand all
434434
435435 /// Advance \p It while it points to a debug instruction and return the result.
436436 /// This assumes that \p It is not at the end of a block.
437 BasicBlock::iterator skipDebugInfo(BasicBlock::iterator It);
437 BasicBlock::iterator skipDebugIntrinsics(BasicBlock::iterator It);
438438
439439 } // end namespace llvm
440440
479479 return Optional();
480480 }
481481
482 BasicBlock::iterator llvm::skipDebugInfo(BasicBlock::iterator It) {
482 BasicBlock::iterator llvm::skipDebugIntrinsics(BasicBlock::iterator It) {
483483 while (isa(It))
484484 ++It;
485485 return It;
871871 // The first non-debug instruction is the terminator.
872872 auto *Term = BB.getTerminator();
873873 EXPECT_EQ(Term, BB.begin()->getNextNonDebugInstruction());
874 EXPECT_EQ(Term->getIterator(), skipDebugInfo(BB.begin()));
874 EXPECT_EQ(Term->getIterator(), skipDebugIntrinsics(BB.begin()));
875875
876876 // After the terminator, there are no non-debug instructions.
877877 EXPECT_EQ(nullptr, Term->getNextNonDebugInstruction());