diff options
author | Rafael Espindola <rafael.espindola@gmail.com> | 2013-07-03 15:46:03 +0000 |
---|---|---|
committer | Rafael Espindola <rafael.espindola@gmail.com> | 2013-07-03 15:46:03 +0000 |
commit | 40d40dd44edbac3c20a4f3305f208ab48c2c219a (patch) | |
tree | f5782d0be8e4d05ddb0bbe26f994ef44c5141539 | |
parent | f6b67dc7f8ed87443dc03856e789f42ba72ecaa8 (diff) | |
download | llvm-40d40dd44edbac3c20a4f3305f208ab48c2c219a.tar.gz llvm-40d40dd44edbac3c20a4f3305f208ab48c2c219a.tar.bz2 llvm-40d40dd44edbac3c20a4f3305f208ab48c2c219a.tar.xz |
Make llvm-nm return 1 on error.
This is a small compatibility improvement with gnu nm and makes llvm-nm more
useful as a testing tool.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@185546 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | test/Object/nm-error.test | 17 | ||||
-rw-r--r-- | tools/llvm-nm/llvm-nm.cpp | 8 |
2 files changed, 25 insertions, 0 deletions
diff --git a/test/Object/nm-error.test b/test/Object/nm-error.test new file mode 100644 index 0000000000..146b88713f --- /dev/null +++ b/test/Object/nm-error.test @@ -0,0 +1,17 @@ +Test that llvm-nm returns an error because of the unknown file type, but +keeps processing subsequent files. + +Note: We use a temporary file since the tests don't run with pipefail. + +RUN: touch %t +RUN: not llvm-nm %p/Inputs/trivial-object-test.elf-i386 %t \ +RUN: %p/Inputs/trivial-object-test.elf-i386 > %t.log +RUN: FileCheck %s < %t.log + +CHECK: U SomeOtherFunction +CHECK: 00000000 T main +CHECK: U puts + +CHECK: U SomeOtherFunction +CHECK: 00000000 T main +CHECK: U puts diff --git a/tools/llvm-nm/llvm-nm.cpp b/tools/llvm-nm/llvm-nm.cpp index cb46520763..c33289bd87 100644 --- a/tools/llvm-nm/llvm-nm.cpp +++ b/tools/llvm-nm/llvm-nm.cpp @@ -121,6 +121,8 @@ namespace { bool MultipleFiles = false; + bool HadError = false; + std::string ToolName; } @@ -132,6 +134,7 @@ static void error(Twine message, Twine path = Twine()) { static bool error(error_code ec, Twine path = Twine()) { if (ec) { error(ec.message(), path); + HadError = true; return true; } return false; @@ -429,6 +432,7 @@ static void DumpSymbolNamesFromFile(std::string &Filename) { } else { errs() << ToolName << ": " << Filename << ": " << "unrecognizable file type\n"; + HadError = true; return; } } @@ -463,5 +467,9 @@ int main(int argc, char **argv) { std::for_each(InputFilenames.begin(), InputFilenames.end(), DumpSymbolNamesFromFile); + + if (HadError) + return 1; + return 0; } |