diff options
| author | Vedant Kumar <vsk@apple.com> | 2018-06-26 21:16:59 +0000 |
|---|---|---|
| committer | Vedant Kumar <vsk@apple.com> | 2018-06-26 21:16:59 +0000 |
| commit | 1cb63dc2d59233ac71b4399b6f90d3208c50a791 (patch) | |
| tree | be01177cb486756fa17c8cc853bbf83ae5b61dd4 /llvm/unittests/IR | |
| parent | 70ec1dd14d66a088be290908d2985652b4863892 (diff) | |
| download | bcm5719-llvm-1cb63dc2d59233ac71b4399b6f90d3208c50a791.tar.gz bcm5719-llvm-1cb63dc2d59233ac71b4399b6f90d3208c50a791.zip | |
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!
llvm-svn: 335667
Diffstat (limited to 'llvm/unittests/IR')
| -rw-r--r-- | llvm/unittests/IR/InstructionsTest.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/unittests/IR/InstructionsTest.cpp b/llvm/unittests/IR/InstructionsTest.cpp index 69e74544a90..6bac59620d9 100644 --- a/llvm/unittests/IR/InstructionsTest.cpp +++ b/llvm/unittests/IR/InstructionsTest.cpp @@ -872,7 +872,7 @@ TEST(InstructionsTest, SkipDebug) { // The first non-debug instruction is the terminator. auto *Term = BB.getTerminator(); EXPECT_EQ(Term, BB.begin()->getNextNonDebugInstruction()); - EXPECT_EQ(Term->getIterator(), skipDebugInfo(BB.begin())); + EXPECT_EQ(Term->getIterator(), skipDebugIntrinsics(BB.begin())); // After the terminator, there are no non-debug instructions. EXPECT_EQ(nullptr, Term->getNextNonDebugInstruction()); |

