[Lldb-commits] [lldb] r235584 - Fix test expectation in TestNoreturnUnwind
Tamas Berghammer
tberghammer at google.com
Thu Apr 23 03:54:28 PDT 2015
Author: tberghammer
Date: Thu Apr 23 05:54:27 2015
New Revision: 235584
URL: http://llvm.org/viewvc/llvm-project?rev=235584&view=rev
Log:
Fix test expectation in TestNoreturnUnwind
The test case lookinhg for the abort function in the stack trace.
Previously it lookd for a function which ends with "abort" but on some
system there are multiple such functions (e.g.: on android abort calls
__libc_android_abort) what made the test fail. This CL change the
behaviour to look for the abort function based on a fix list of names.
Modified:
lldb/trunk/test/functionalities/unwind/noreturn/TestNoreturnUnwind.py
Modified: lldb/trunk/test/functionalities/unwind/noreturn/TestNoreturnUnwind.py
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/test/functionalities/unwind/noreturn/TestNoreturnUnwind.py?rev=235584&r1=235583&r2=235584&view=diff
==============================================================================
--- lldb/trunk/test/functionalities/unwind/noreturn/TestNoreturnUnwind.py (original)
+++ lldb/trunk/test/functionalities/unwind/noreturn/TestNoreturnUnwind.py Thu Apr 23 05:54:27 2015
@@ -45,9 +45,8 @@ class NoreturnUnwind(TestBase):
thread = process.GetThreadAtIndex(0)
abort_frame_number = 0
for f in thread.frames:
- # We use endswith() to look for abort() since some C libraries mangle the symbol into
- # __GI_abort or similar.
- if f.GetFunctionName().endswith("abort"):
+ # Some C libraries mangle the abort symbol into __GI_abort.
+ if f.GetFunctionName() in ["abort", "__GI_abort"]:
break
abort_frame_number = abort_frame_number + 1
More information about the lldb-commits
mailing list