diff options
| author | James Henderson <jh7370@my.bristol.ac.uk> | 2019-03-12 15:42:38 +0000 |
|---|---|---|
| committer | James Henderson <jh7370@my.bristol.ac.uk> | 2019-03-12 15:42:38 +0000 |
| commit | b69a50115bd5db998dd972d2017b2332d4033b5c (patch) | |
| tree | cf32bb22994f7bcd2b905307cf1a1a275cbd21f7 | |
| parent | 662c043628c1dfe9765f3f323cc57931c6d464d4 (diff) | |
| download | bcm5719-llvm-b69a50115bd5db998dd972d2017b2332d4033b5c.tar.gz bcm5719-llvm-b69a50115bd5db998dd972d2017b2332d4033b5c.zip | |
[llvm-cxxfilt]Add test to show that empty lines can be handled
I recently discovered a bug in llvm-cxxfilt introduced in r353743 but
was fixed later incidentally due to r355031. Specifically, llvm-cxxfilt
was attempting to call .back() on an empty string any time there was a
new line in the input. This was causing a crash in my debug builds only.
This patch simply adds a test that explicitly tests that llvm-cxxfilt
handles empty lines correctly. It may pass under release builds under
the broken behaviour, but it fails at least in debug builds.
Reviewed by: mattd
Differential Revision: https://reviews.llvm.org/D58785
llvm-svn: 355929
| -rw-r--r-- | llvm/test/tools/llvm-cxxfilt/empty-line.test | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/llvm/test/tools/llvm-cxxfilt/empty-line.test b/llvm/test/tools/llvm-cxxfilt/empty-line.test new file mode 100644 index 00000000000..1b62ee0812f --- /dev/null +++ b/llvm/test/tools/llvm-cxxfilt/empty-line.test @@ -0,0 +1,9 @@ +# Show that llvm-cxxfilt can handle empty lines in the input. +RUN: echo "" > %t +RUN: echo "_Z3foov" >> %t +RUN: echo "" >> %t +RUN: llvm-cxxfilt < %t | FileCheck %s + +CHECK: {{^$}} +CHECK-NEXT: foo() +CHECK-EMPTY: |
