[llvm-cxxfilt]Add test to show that empty lines can be handled
authorJames Henderson <jh7370@my.bristol.ac.uk>
Tue, 12 Mar 2019 15:42:38 +0000 (15:42 +0000)
committerJames Henderson <jh7370@my.bristol.ac.uk>
Tue, 12 Mar 2019 15:42:38 +0000 (15:42 +0000)
commitb69a50115bd5db998dd972d2017b2332d4033b5c
treecf32bb22994f7bcd2b905307cf1a1a275cbd21f7
parent662c043628c1dfe9765f3f323cc57931c6d464d4
[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
llvm/test/tools/llvm-cxxfilt/empty-line.test [new file with mode: 0644]