[Lldb-commits] [lldb] a06342d - [lldb] Modify TestThreadJump to work around a change in clang's debug_line generation

Pavel Labath via lldb-commits lldb-commits at lists.llvm.org
Mon Dec 12 00:31:55 PST 2022


Author: Pavel Labath
Date: 2022-12-12T09:31:35+01:00
New Revision: a06342d250ec7bee37dc93477f233e43e597aca5

URL: https://github.com/llvm/llvm-project/commit/a06342d250ec7bee37dc93477f233e43e597aca5
DIFF: https://github.com/llvm/llvm-project/commit/a06342d250ec7bee37dc93477f233e43e597aca5.diff

LOG: [lldb] Modify TestThreadJump to work around a change in clang's debug_line generation

After D133376, jumping to the return line in the otherfn function became
ambiguous because it has two line entries associated with it. Work
around that problem by changing the function. Filed PR59458 to track
possible improvements in jump target disambiguation.

Added: 
    

Modified: 
    lldb/test/API/functionalities/thread/jump/other.cpp

Removed: 
    


################################################################################
diff  --git a/lldb/test/API/functionalities/thread/jump/other.cpp b/lldb/test/API/functionalities/thread/jump/other.cpp
index 741f13bbd5221..c44786f721d57 100644
--- a/lldb/test/API/functionalities/thread/jump/other.cpp
+++ b/lldb/test/API/functionalities/thread/jump/other.cpp
@@ -1,4 +1,4 @@
-int otherfn()
-{
-    return 4; // other marker
+int otherfn() {
+  int x = 4; // other marker
+  return x;
 }


        


More information about the lldb-commits mailing list