This revision was automatically updated to reflect the committed changes.
Closed by commit rGbddd28882644: [lldb/DWARF] Add support for pre-standard GNU
call site attributes (authored by labath).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D80519/new/
https://reviews.llvm.org/D80519
Files:
lldb/source/Expression/DWARFExpression.cpp
lldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp
lldb/test/API/functionalities/param_entry_vals/basic_entry_values/TestBasicEntryValues.py
lldb/test/API/functionalities/param_entry_vals/basic_entry_values/main.cpp
lldb/test/API/functionalities/tail_call_frames/ambiguous_tail_call_seq1/Makefile
lldb/test/API/functionalities/tail_call_frames/ambiguous_tail_call_seq1/TestAmbiguousTailCallSeq1.py
lldb/test/API/functionalities/tail_call_frames/ambiguous_tail_call_seq2/Makefile
lldb/test/API/functionalities/tail_call_frames/ambiguous_tail_call_seq2/TestAmbiguousTailCallSeq2.py
lldb/test/API/functionalities/tail_call_frames/cross_dso/Makefile
lldb/test/API/functionalities/tail_call_frames/cross_dso/One.mk
lldb/test/API/functionalities/tail_call_frames/cross_dso/Two.mk
lldb/test/API/functionalities/tail_call_frames/cross_object/Makefile
lldb/test/API/functionalities/tail_call_frames/disambiguate_call_site/Makefile
lldb/test/API/functionalities/tail_call_frames/disambiguate_call_site/TestDisambiguateCallSite.py
lldb/test/API/functionalities/tail_call_frames/disambiguate_call_site/main.cpp
lldb/test/API/functionalities/tail_call_frames/disambiguate_paths_to_common_sink/Makefile
lldb/test/API/functionalities/tail_call_frames/disambiguate_paths_to_common_sink/TestDisambiguatePathsToCommonSink.py
lldb/test/API/functionalities/tail_call_frames/disambiguate_paths_to_common_sink/main.cpp
lldb/test/API/functionalities/tail_call_frames/disambiguate_tail_call_seq/Makefile
lldb/test/API/functionalities/tail_call_frames/disambiguate_tail_call_seq/TestDisambiguateTailCallSeq.py
lldb/test/API/functionalities/tail_call_frames/disambiguate_tail_call_seq/main.cpp
lldb/test/API/functionalities/tail_call_frames/inlining_and_tail_calls/Makefile
lldb/test/API/functionalities/tail_call_frames/inlining_and_tail_calls/TestInliningAndTailCalls.py
lldb/test/API/functionalities/tail_call_frames/inlining_and_tail_calls/main.cpp
lldb/test/API/functionalities/tail_call_frames/sbapi_support/Makefile
lldb/test/API/functionalities/tail_call_frames/thread_step_out_message/Makefile
lldb/test/API/functionalities/tail_call_frames/thread_step_out_message/TestArtificialFrameStepOutMessage.py
lldb/test/API/functionalities/tail_call_frames/thread_step_out_message/main.cpp
lldb/test/API/functionalities/tail_call_frames/thread_step_out_or_return/Makefile
lldb/test/API/functionalities/tail_call_frames/unambiguous_sequence/Makefile
lldb/test/API/functionalities/tail_call_frames/unambiguous_sequence/TestUnambiguousTailCalls.py
lldb/test/API/functionalities/tail_call_frames/unambiguous_sequence/main.cpp
Index: lldb/test/API/functionalities/tail_call_frames/unambiguous_sequence/main.cpp
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/unambiguous_sequence/main.cpp
+++ lldb/test/API/functionalities/tail_call_frames/unambiguous_sequence/main.cpp
@@ -2,11 +2,13 @@
void __attribute__((noinline)) sink() {
x++; //% self.filecheck("bt", "main.cpp", "-implicit-check-not=artificial")
- // CHECK: frame #0: 0x{{[0-9a-f]+}} a.out`sink() at main.cpp:[[@LINE-1]]:4 [opt]
- // CHECK-NEXT: frame #1: 0x{{[0-9a-f]+}} a.out`func3() at main.cpp:14:3 [opt] [artificial]
- // CHECK-NEXT: frame #2: 0x{{[0-9a-f]+}} a.out`func2() {{.*}} [opt]
- // CHECK-NEXT: frame #3: 0x{{[0-9a-f]+}} a.out`func1() at main.cpp:23:3 [opt] [artificial]
- // CHECK-NEXT: frame #4: 0x{{[0-9a-f]+}} a.out`main{{.*}} [opt]
+ // CHECK: frame #0: 0x{{[0-9a-f]+}} a.out`sink() at main.cpp:[[@LINE-1]]:4
+ // CHECK-NEXT: frame #1: 0x{{[0-9a-f]+}} a.out`func3() at main.cpp:16:3
+ // CHECK-SAME: [artificial]
+ // CHECK-NEXT: frame #2: 0x{{[0-9a-f]+}} a.out`func2()
+ // CHECK-NEXT: frame #3: 0x{{[0-9a-f]+}} a.out`func1() at main.cpp:25:3
+ // CHECK-SAME: [artificial]
+ // CHECK-NEXT: frame #4: 0x{{[0-9a-f]+}} a.out`main
}
void __attribute__((noinline)) func3() {
Index: lldb/test/API/functionalities/tail_call_frames/unambiguous_sequence/TestUnambiguousTailCalls.py
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/unambiguous_sequence/TestUnambiguousTailCalls.py
+++ lldb/test/API/functionalities/tail_call_frames/unambiguous_sequence/TestUnambiguousTailCalls.py
@@ -1,6 +1,9 @@
from lldbsuite.test import lldbinline
from lldbsuite.test import decorators
-lldbinline.MakeInlineTest(__file__, globals(),
- [decorators.skipUnlessHasCallSiteInfo,
- decorators.skipIf(dwarf_version=['<', '4'])])
+decorators = [decorators.skipUnlessHasCallSiteInfo,
+ decorators.skipIf(dwarf_version=['<', '4'])]
+lldbinline.MakeInlineTest(__file__, globals(), name="UnambiguousTailCalls_V5",
+ build_dict=dict(CFLAGS_EXTRAS="-O2 -glldb"), decorators=decorators)
+lldbinline.MakeInlineTest(__file__, globals(), name="UnambiguousTailCalls_GNU",
+ build_dict=dict(CFLAGS_EXTRAS="-O2 -ggdb"), decorators=decorators)
Index: lldb/test/API/functionalities/tail_call_frames/unambiguous_sequence/Makefile
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/unambiguous_sequence/Makefile
+++ /dev/null
@@ -1,4 +0,0 @@
-CXX_SOURCES := main.cpp
-
-CXXFLAGS_EXTRAS := -g -O2 -glldb
-include Makefile.rules
Index: lldb/test/API/functionalities/tail_call_frames/thread_step_out_or_return/Makefile
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/thread_step_out_or_return/Makefile
+++ lldb/test/API/functionalities/tail_call_frames/thread_step_out_or_return/Makefile
@@ -1,4 +1,4 @@
CXX_SOURCES := main.cpp
-CXXFLAGS_EXTRAS := -g -O2 -glldb
+CXXFLAGS_EXTRAS := -O2
include Makefile.rules
Index: lldb/test/API/functionalities/tail_call_frames/thread_step_out_message/main.cpp
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/thread_step_out_message/main.cpp
+++ lldb/test/API/functionalities/tail_call_frames/thread_step_out_message/main.cpp
@@ -3,8 +3,8 @@
void __attribute__((noinline)) sink() {
x++; //% self.filecheck("finish", "main.cpp", "-implicit-check-not=artificial")
// CHECK: stop reason = step out
- // CHECK-NEXT: Stepped out past: frame #1: 0x{{[0-9a-f]+}} a.out`func3{{.*}} [opt] [artificial]
- // CHECK: frame #0: 0x{{[0-9a-f]+}} a.out`func2{{.*}} [opt]
+ // CHECK-NEXT: Stepped out past: frame #1: 0x{{[0-9a-f]+}} a.out`func3{{.*}} [artificial]
+ // CHECK: frame #0: 0x{{[0-9a-f]+}} a.out`func2{{.*}}
}
void __attribute__((noinline)) func3() { sink(); /* tail */ }
Index: lldb/test/API/functionalities/tail_call_frames/thread_step_out_message/TestArtificialFrameStepOutMessage.py
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/thread_step_out_message/TestArtificialFrameStepOutMessage.py
+++ lldb/test/API/functionalities/tail_call_frames/thread_step_out_message/TestArtificialFrameStepOutMessage.py
@@ -1,6 +1,11 @@
from lldbsuite.test import lldbinline
from lldbsuite.test import decorators
+decorators = [decorators.skipUnlessHasCallSiteInfo,
+ decorators.skipIf(dwarf_version=['<', '4'])]
lldbinline.MakeInlineTest(__file__, globals(),
- [decorators.skipUnlessHasCallSiteInfo,
- decorators.skipIf(dwarf_version=['<', '4'])])
+ name="ArtificialFrameStepOutMessage_V5",
+ build_dict=dict(CFLAGS_EXTRAS="-O2 -glldb"), decorators=decorators)
+lldbinline.MakeInlineTest(__file__, globals(),
+ name="ArtificialFrameStepOutMessage_GNU",
+ build_dict=dict(CFLAGS_EXTRAS="-O2 -ggdb"), decorators=decorators)
Index: lldb/test/API/functionalities/tail_call_frames/thread_step_out_message/Makefile
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/thread_step_out_message/Makefile
+++ /dev/null
@@ -1,4 +0,0 @@
-CXX_SOURCES := main.cpp
-
-CXXFLAGS_EXTRAS := -g -O2 -glldb
-include Makefile.rules
Index: lldb/test/API/functionalities/tail_call_frames/sbapi_support/Makefile
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/sbapi_support/Makefile
+++ lldb/test/API/functionalities/tail_call_frames/sbapi_support/Makefile
@@ -1,4 +1,4 @@
CXX_SOURCES := main.cpp
-CXXFLAGS_EXTRAS := -g -O2 -glldb
+CXXFLAGS_EXTRAS := -O2
include Makefile.rules
Index: lldb/test/API/functionalities/tail_call_frames/inlining_and_tail_calls/main.cpp
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/inlining_and_tail_calls/main.cpp
+++ lldb/test/API/functionalities/tail_call_frames/inlining_and_tail_calls/main.cpp
@@ -2,9 +2,9 @@
void __attribute__((noinline)) tail_call_sink() {
x++; //% self.filecheck("bt", "main.cpp", "-check-prefix=TAIL-CALL-SINK")
- // TAIL-CALL-SINK: frame #0: 0x{{[0-9a-f]+}} a.out`tail_call_sink() at main.cpp:[[@LINE-1]]:4 [opt]
- // TAIL-CALL-SINK-NEXT: func3{{.*}} [opt] [artificial]
- // TAIL-CALL-SINK-NEXT: main{{.*}} [opt]
+ // TAIL-CALL-SINK: frame #0: 0x{{[0-9a-f]+}} a.out`tail_call_sink() at main.cpp:[[@LINE-1]]:4
+ // TAIL-CALL-SINK-NEXT: func3{{.*}} [artificial]
+ // TAIL-CALL-SINK-NEXT: main{{.*}}
// TODO: The backtrace should include inlinable_function_which_tail_calls.
}
@@ -19,10 +19,10 @@
void __attribute__((always_inline)) inline_sink() {
x++; //% self.filecheck("bt", "main.cpp", "-check-prefix=INLINE-SINK")
- // INLINE-SINK: frame #0: 0x{{[0-9a-f]+}} a.out`func2() [inlined] inline_sink() at main.cpp:[[@LINE-1]]:4 [opt]
- // INLINE-SINK-NEXT: func2{{.*}} [opt]
- // INLINE-SINK-NEXT: func1{{.*}} [opt] [artificial]
- // INLINE-SINK-NEXT: main{{.*}} [opt]
+ // INLINE-SINK: frame #0: 0x{{[0-9a-f]+}} a.out`func2() [inlined] inline_sink() at main.cpp:[[@LINE-1]]:4
+ // INLINE-SINK-NEXT: func2{{.*}}
+ // INLINE-SINK-NEXT: func1{{.*}} [artificial]
+ // INLINE-SINK-NEXT: main{{.*}}
}
void __attribute__((noinline)) func2() { inline_sink(); /* inlined */ }
Index: lldb/test/API/functionalities/tail_call_frames/inlining_and_tail_calls/TestInliningAndTailCalls.py
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/inlining_and_tail_calls/TestInliningAndTailCalls.py
+++ lldb/test/API/functionalities/tail_call_frames/inlining_and_tail_calls/TestInliningAndTailCalls.py
@@ -1,6 +1,9 @@
from lldbsuite.test import lldbinline
from lldbsuite.test import decorators
-lldbinline.MakeInlineTest(__file__, globals(),
- [decorators.skipUnlessHasCallSiteInfo,
- decorators.skipIf(dwarf_version=['<', '4'])])
+decorators = [decorators.skipUnlessHasCallSiteInfo,
+ decorators.skipIf(dwarf_version=['<', '4'])]
+lldbinline.MakeInlineTest(__file__, globals(), name="InliningAndTailCalls_V5",
+ build_dict=dict(CFLAGS_EXTRAS="-O2 -glldb"), decorators=decorators)
+lldbinline.MakeInlineTest(__file__, globals(), name="InliningAndTailCalls_GNU",
+ build_dict=dict(CFLAGS_EXTRAS="-O2 -ggdb"), decorators=decorators)
Index: lldb/test/API/functionalities/tail_call_frames/inlining_and_tail_calls/Makefile
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/inlining_and_tail_calls/Makefile
+++ /dev/null
@@ -1,4 +0,0 @@
-CXX_SOURCES := main.cpp
-
-CXXFLAGS_EXTRAS := -g -O2 -glldb
-include Makefile.rules
Index: lldb/test/API/functionalities/tail_call_frames/disambiguate_tail_call_seq/main.cpp
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/disambiguate_tail_call_seq/main.cpp
+++ lldb/test/API/functionalities/tail_call_frames/disambiguate_tail_call_seq/main.cpp
@@ -2,10 +2,10 @@
void __attribute__((noinline)) sink() {
x++; //% self.filecheck("bt", "main.cpp", "-implicit-check-not=artificial")
- // CHECK: frame #0: 0x{{[0-9a-f]+}} a.out`sink() at main.cpp:[[@LINE-1]]:4 [opt]
- // CHECK-NEXT: func3{{.*}} [opt] [artificial]
- // CHECK-NEXT: func1{{.*}} [opt] [artificial]
- // CHECK-NEXT: main{{.*}} [opt]
+ // CHECK: frame #0: 0x{{[0-9a-f]+}} a.out`sink() at main.cpp:[[@LINE-1]]:4
+ // CHECK-NEXT: func3{{.*}} [artificial]
+ // CHECK-NEXT: func1{{.*}} [artificial]
+ // CHECK-NEXT: main{{.*}}
}
void __attribute__((noinline)) func3() { sink(); /* tail */ }
Index: lldb/test/API/functionalities/tail_call_frames/disambiguate_tail_call_seq/TestDisambiguateTailCallSeq.py
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/disambiguate_tail_call_seq/TestDisambiguateTailCallSeq.py
+++ lldb/test/API/functionalities/tail_call_frames/disambiguate_tail_call_seq/TestDisambiguateTailCallSeq.py
@@ -1,6 +1,9 @@
from lldbsuite.test import lldbinline
from lldbsuite.test import decorators
-lldbinline.MakeInlineTest(__file__, globals(),
- [decorators.skipUnlessHasCallSiteInfo,
- decorators.skipIf(dwarf_version=['<', '4'])])
+decorators = [decorators.skipUnlessHasCallSiteInfo,
+ decorators.skipIf(dwarf_version=['<', '4'])]
+lldbinline.MakeInlineTest(__file__, globals(), name="DisambiguateTailCallSeq_V5",
+ build_dict=dict(CFLAGS_EXTRAS="-O2 -glldb"), decorators=decorators)
+lldbinline.MakeInlineTest(__file__, globals(), name="DisambiguateTailCallSeq_GNU",
+ build_dict=dict(CFLAGS_EXTRAS="-O2 -ggdb"), decorators=decorators)
Index: lldb/test/API/functionalities/tail_call_frames/disambiguate_tail_call_seq/Makefile
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/disambiguate_tail_call_seq/Makefile
+++ /dev/null
@@ -1,4 +0,0 @@
-CXX_SOURCES := main.cpp
-
-CXXFLAGS_EXTRAS := -g -O2 -glldb
-include Makefile.rules
Index: lldb/test/API/functionalities/tail_call_frames/disambiguate_paths_to_common_sink/main.cpp
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/disambiguate_paths_to_common_sink/main.cpp
+++ lldb/test/API/functionalities/tail_call_frames/disambiguate_paths_to_common_sink/main.cpp
@@ -2,17 +2,18 @@
void __attribute__((noinline)) sink2() {
x++; //% self.filecheck("bt", "main.cpp", "-check-prefix=FROM-FUNC1")
- // FROM-FUNC1: frame #0: 0x{{[0-9a-f]+}} a.out`sink{{.*}} at main.cpp:[[@LINE-1]]:{{.*}} [opt]
- // FROM-FUNC1-NEXT: sink({{.*}} [opt]
- // FROM-FUNC1-NEXT: func1{{.*}} [opt] [artificial]
- // FROM-FUNC1-NEXT: main{{.*}} [opt]
+ // FROM-FUNC1: frame #0: 0x{{[0-9a-f]+}} a.out`sink{{.*}} at main.cpp:[[@LINE-1]]
+ // FROM-FUNC1-NEXT: sink
+ // FROM-FUNC1-NEXT: func1
+ // FROM-FUNC1-SAME: [artificial]
+ // FROM-FUNC1-NEXT: main
}
void __attribute__((noinline)) sink(bool called_from_main) {
if (called_from_main) {
x++; //% self.filecheck("bt", "main.cpp", "-check-prefix=FROM-MAIN")
- // FROM-MAIN: frame #0: 0x{{[0-9a-f]+}} a.out`sink{{.*}} at main.cpp:[[@LINE-1]]:{{.*}} [opt]
- // FROM-MAIN-NEXT: main{{.*}} [opt]
+ // FROM-MAIN: frame #0: 0x{{[0-9a-f]+}} a.out`sink{{.*}} at main.cpp:[[@LINE-1]]
+ // FROM-MAIN-NEXT: main
} else {
sink2();
}
Index: lldb/test/API/functionalities/tail_call_frames/disambiguate_paths_to_common_sink/TestDisambiguatePathsToCommonSink.py
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/disambiguate_paths_to_common_sink/TestDisambiguatePathsToCommonSink.py
+++ lldb/test/API/functionalities/tail_call_frames/disambiguate_paths_to_common_sink/TestDisambiguatePathsToCommonSink.py
@@ -1,6 +1,11 @@
from lldbsuite.test import lldbinline
from lldbsuite.test import decorators
+decorators = [decorators.skipUnlessHasCallSiteInfo,
+ decorators.skipIf(dwarf_version=['<', '4'])]
lldbinline.MakeInlineTest(__file__, globals(),
- [decorators.skipUnlessHasCallSiteInfo,
- decorators.skipIf(dwarf_version=['<', '4'])])
+ name="DisambiguatePathsToCommonSink_V5",
+ build_dict=dict(CFLAGS_EXTRAS="-O2 -glldb"), decorators=decorators)
+lldbinline.MakeInlineTest(__file__, globals(),
+ name="DisambiguatePathsToCommonSink_GNU",
+ build_dict=dict(CFLAGS_EXTRAS="-O2 -ggdb"), decorators=decorators)
Index: lldb/test/API/functionalities/tail_call_frames/disambiguate_paths_to_common_sink/Makefile
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/disambiguate_paths_to_common_sink/Makefile
+++ /dev/null
@@ -1,4 +0,0 @@
-CXX_SOURCES := main.cpp
-
-CXXFLAGS_EXTRAS := -g -O2 -glldb
-include Makefile.rules
Index: lldb/test/API/functionalities/tail_call_frames/disambiguate_call_site/main.cpp
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/disambiguate_call_site/main.cpp
+++ lldb/test/API/functionalities/tail_call_frames/disambiguate_call_site/main.cpp
@@ -2,9 +2,9 @@
void __attribute__((noinline)) sink() {
x++; //% self.filecheck("bt", "main.cpp", "-implicit-check-not=artificial")
- // CHECK: frame #0: 0x{{[0-9a-f]+}} a.out`sink() at main.cpp:[[@LINE-1]]:4 [opt]
- // CHECK-NEXT: func2{{.*}} [opt] [artificial]
- // CHECK-NEXT: main{{.*}} [opt]
+ // CHECK: frame #0: 0x{{[0-9a-f]+}} a.out`sink() at main.cpp:[[@LINE-1]]:4
+ // CHECK-NEXT: func2{{.*}} [artificial]
+ // CHECK-NEXT: main{{.*}}
}
void __attribute__((noinline)) func2() {
Index: lldb/test/API/functionalities/tail_call_frames/disambiguate_call_site/TestDisambiguateCallSite.py
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/disambiguate_call_site/TestDisambiguateCallSite.py
+++ lldb/test/API/functionalities/tail_call_frames/disambiguate_call_site/TestDisambiguateCallSite.py
@@ -1,6 +1,9 @@
from lldbsuite.test import lldbinline
from lldbsuite.test import decorators
-lldbinline.MakeInlineTest(__file__, globals(),
- [decorators.skipUnlessHasCallSiteInfo,
- decorators.skipIf(dwarf_version=['<', '4'])])
+decorators = [decorators.skipUnlessHasCallSiteInfo,
+ decorators.skipIf(dwarf_version=['<', '4'])]
+lldbinline.MakeInlineTest(__file__, globals(), name="DisambiguateCallSite_V5",
+ build_dict=dict(CFLAGS_EXTRAS="-O2 -glldb"), decorators=decorators)
+lldbinline.MakeInlineTest(__file__, globals(), name="DisambiguateCallSite_GNU",
+ build_dict=dict(CFLAGS_EXTRAS="-O2 -ggdb"), decorators=decorators)
Index: lldb/test/API/functionalities/tail_call_frames/disambiguate_call_site/Makefile
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/disambiguate_call_site/Makefile
+++ /dev/null
@@ -1,4 +0,0 @@
-CXX_SOURCES := main.cpp
-
-CXXFLAGS_EXTRAS := -g -O2 -glldb
-include Makefile.rules
Index: lldb/test/API/functionalities/tail_call_frames/cross_object/Makefile
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/cross_object/Makefile
+++ lldb/test/API/functionalities/tail_call_frames/cross_object/Makefile
@@ -1,4 +1,4 @@
C_SOURCES := main.c One.c Two.c
-CFLAGS_EXTRAS := -g -O2 -glldb
+CFLAGS_EXTRAS := -O2
include Makefile.rules
Index: lldb/test/API/functionalities/tail_call_frames/cross_dso/Two.mk
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/cross_dso/Two.mk
+++ lldb/test/API/functionalities/tail_call_frames/cross_dso/Two.mk
@@ -1,6 +1,6 @@
DYLIB_NAME := Two
DYLIB_C_SOURCES := Two.c
DYLIB_ONLY := YES
-CFLAGS_EXTRAS := -g -O2 -glldb
+CFLAGS_EXTRAS := -O2
include Makefile.rules
Index: lldb/test/API/functionalities/tail_call_frames/cross_dso/One.mk
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/cross_dso/One.mk
+++ lldb/test/API/functionalities/tail_call_frames/cross_dso/One.mk
@@ -1,7 +1,7 @@
DYLIB_NAME := One
DYLIB_C_SOURCES := One.c
DYLIB_ONLY := YES
-CFLAGS_EXTRAS := -g -O2 -glldb
+CFLAGS_EXTRAS := -O2
LD_EXTRAS := -L. -lTwo
include Makefile.rules
Index: lldb/test/API/functionalities/tail_call_frames/cross_dso/Makefile
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/cross_dso/Makefile
+++ lldb/test/API/functionalities/tail_call_frames/cross_dso/Makefile
@@ -1,6 +1,6 @@
LD_EXTRAS := -L. -l$(LIB_PREFIX)One -l$(LIB_PREFIX)Two
C_SOURCES := main.c
-CFLAGS_EXTRAS := -g -O2 -glldb
+CFLAGS_EXTRAS := -O2
include Makefile.rules
Index: lldb/test/API/functionalities/tail_call_frames/ambiguous_tail_call_seq2/TestAmbiguousTailCallSeq2.py
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/ambiguous_tail_call_seq2/TestAmbiguousTailCallSeq2.py
+++ lldb/test/API/functionalities/tail_call_frames/ambiguous_tail_call_seq2/TestAmbiguousTailCallSeq2.py
@@ -1,6 +1,9 @@
from lldbsuite.test import lldbinline
from lldbsuite.test import decorators
-lldbinline.MakeInlineTest(__file__, globals(),
- [decorators.skipUnlessHasCallSiteInfo,
- decorators.skipIf(dwarf_version=['<', '4'])])
+decorators = [decorators.skipUnlessHasCallSiteInfo,
+ decorators.skipIf(dwarf_version=['<', '4'])]
+lldbinline.MakeInlineTest(__file__, globals(), name="AmbiguousTailCallSeq2_V5",
+ build_dict=dict(CFLAGS_EXTRAS="-O2 -glldb"), decorators=decorators)
+lldbinline.MakeInlineTest(__file__, globals(), name="AmbiguousTailCallSeq2_GNU",
+ build_dict=dict(CFLAGS_EXTRAS="-O2 -ggdb"), decorators=decorators)
Index: lldb/test/API/functionalities/tail_call_frames/ambiguous_tail_call_seq2/Makefile
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/ambiguous_tail_call_seq2/Makefile
+++ /dev/null
@@ -1,4 +0,0 @@
-CXX_SOURCES := main.cpp
-
-CXXFLAGS_EXTRAS := -g -O2 -glldb
-include Makefile.rules
Index: lldb/test/API/functionalities/tail_call_frames/ambiguous_tail_call_seq1/TestAmbiguousTailCallSeq1.py
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/ambiguous_tail_call_seq1/TestAmbiguousTailCallSeq1.py
+++ lldb/test/API/functionalities/tail_call_frames/ambiguous_tail_call_seq1/TestAmbiguousTailCallSeq1.py
@@ -1,6 +1,9 @@
from lldbsuite.test import lldbinline
from lldbsuite.test import decorators
-lldbinline.MakeInlineTest(__file__, globals(),
- [decorators.skipUnlessHasCallSiteInfo,
- decorators.skipIf(dwarf_version=['<', '4'])])
+decorators = [decorators.skipUnlessHasCallSiteInfo,
+ decorators.skipIf(dwarf_version=['<', '4'])]
+lldbinline.MakeInlineTest(__file__, globals(), name="AmbiguousTailCallSeq1_V5",
+ build_dict=dict(CFLAGS_EXTRAS="-O2 -glldb"), decorators=decorators)
+lldbinline.MakeInlineTest(__file__, globals(), name="AmbiguousTailCallSeq1_GNU",
+ build_dict=dict(CFLAGS_EXTRAS="-O2 -ggdb"), decorators=decorators)
Index: lldb/test/API/functionalities/tail_call_frames/ambiguous_tail_call_seq1/Makefile
===================================================================
--- lldb/test/API/functionalities/tail_call_frames/ambiguous_tail_call_seq1/Makefile
+++ /dev/null
@@ -1,4 +0,0 @@
-CXX_SOURCES := main.cpp
-
-CXXFLAGS_EXTRAS := -g -O2 -glldb
-include Makefile.rules
Index: lldb/test/API/functionalities/param_entry_vals/basic_entry_values/main.cpp
===================================================================
--- lldb/test/API/functionalities/param_entry_vals/basic_entry_values/main.cpp
+++ lldb/test/API/functionalities/param_entry_vals/basic_entry_values/main.cpp
@@ -18,8 +18,10 @@
use<int &>(dummy);
++global;
- //% self.filecheck("image lookup -v -a $pc", "main.cpp", "-check-prefix=FUNC1-DESC")
- // FUNC1-DESC: name = "sink", type = "int &", location = DW_OP_entry_value
+ //% prefix = "FUNC1-GNU" if "GNU" in self.name else "FUNC1-V5"
+ //% self.filecheck("image lookup -v -a $pc", "main.cpp", "-check-prefix="+prefix)
+ // FUNC1-GNU: name = "sink", type = "int &", location = DW_OP_GNU_entry_value
+ // FUNC1-V5: name = "sink", type = "int &", location = DW_OP_entry_value
}
__attribute__((noinline)) void func2(int &sink, int x) {
Index: lldb/test/API/functionalities/param_entry_vals/basic_entry_values/TestBasicEntryValues.py
===================================================================
--- lldb/test/API/functionalities/param_entry_vals/basic_entry_values/TestBasicEntryValues.py
+++ lldb/test/API/functionalities/param_entry_vals/basic_entry_values/TestBasicEntryValues.py
@@ -11,3 +11,7 @@
lldbinline.MakeInlineTest(__file__, globals(), decorators=decorators,
name="BasicEntryValues_V5",
build_dict=dict(CXXFLAGS_EXTRAS="-O2 -glldb"))
+
+lldbinline.MakeInlineTest(__file__, globals(), decorators=decorators,
+ name="BasicEntryValues_GNU",
+ build_dict=dict(CXXFLAGS_EXTRAS="-O2 -ggdb"))
Index: lldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp
===================================================================
--- lldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp
+++ lldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp
@@ -3601,7 +3601,8 @@
CallSiteParameterArray parameters;
for (DWARFDIE child = call_site_die.GetFirstChild(); child.IsValid();
child = child.GetSibling()) {
- if (child.Tag() != DW_TAG_call_site_parameter)
+ if (child.Tag() != DW_TAG_call_site_parameter &&
+ child.Tag() != DW_TAG_GNU_call_site_parameter)
continue;
llvm::Optional<DWARFExpression> LocationInCallee;
@@ -3631,7 +3632,7 @@
dw_attr_t attr = attributes.AttributeAtIndex(i);
if (attr == DW_AT_location)
LocationInCallee = parse_simple_location(i);
- if (attr == DW_AT_call_value)
+ if (attr == DW_AT_call_value || attr == DW_AT_GNU_call_site_value)
LocationInCaller = parse_simple_location(i);
}
@@ -3648,8 +3649,9 @@
SymbolFileDWARF::CollectCallEdges(ModuleSP module, DWARFDIE function_die) {
// Check if the function has a supported call site-related attribute.
// TODO: In the future it may be worthwhile to support call_all_source_calls.
- uint64_t has_call_edges =
- function_die.GetAttributeValueAsUnsigned(DW_AT_call_all_calls, 0);
+ bool has_call_edges =
+ function_die.GetAttributeValueAsUnsigned(DW_AT_call_all_calls, 0) ||
+ function_die.GetAttributeValueAsUnsigned(DW_AT_GNU_all_call_sites, 0);
if (!has_call_edges)
return {};
@@ -3665,13 +3667,15 @@
std::vector<std::unique_ptr<CallEdge>> call_edges;
for (DWARFDIE child = function_die.GetFirstChild(); child.IsValid();
child = child.GetSibling()) {
- if (child.Tag() != DW_TAG_call_site)
+ if (child.Tag() != DW_TAG_call_site && child.Tag() != DW_TAG_GNU_call_site)
continue;
llvm::Optional<DWARFDIE> call_origin;
llvm::Optional<DWARFExpression> call_target;
addr_t return_pc = LLDB_INVALID_ADDRESS;
addr_t call_inst_pc = LLDB_INVALID_ADDRESS;
+ addr_t low_pc = LLDB_INVALID_ADDRESS;
+ bool tail_call = false;
DWARFAttributes attributes;
const size_t num_attributes = child.GetAttributes(attributes);
@@ -3684,8 +3688,11 @@
dw_attr_t attr = attributes.AttributeAtIndex(i);
+ if (attr == DW_AT_call_tail_call || attr == DW_AT_GNU_tail_call)
+ tail_call = form_value.Boolean();
+
// Extract DW_AT_call_origin (the call target's DIE).
- if (attr == DW_AT_call_origin) {
+ if (attr == DW_AT_call_origin || attr == DW_AT_abstract_origin) {
call_origin = form_value.Reference();
if (!call_origin->IsValid()) {
LLDB_LOG(log, "CollectCallEdges: Invalid call origin in {0}",
@@ -3694,6 +3701,9 @@
}
}
+ if (attr == DW_AT_low_pc)
+ low_pc = form_value.Address();
+
// Extract DW_AT_call_return_pc (the PC the call returns to) if it's
// available. It should only ever be unavailable for tail call edges, in
// which case use LLDB_INVALID_ADDRESS.
@@ -3708,7 +3718,7 @@
// Extract DW_AT_call_target (the location of the address of the indirect
// call).
- if (attr == DW_AT_call_target) {
+ if (attr == DW_AT_call_target || attr == DW_AT_GNU_call_site_target) {
if (!DWARFFormValue::IsBlockForm(form_value.Form())) {
LLDB_LOG(log,
"CollectCallEdges: AT_call_target does not have block form");
@@ -3723,6 +3733,16 @@
child.GetCU());
}
}
+ if (child.Tag() == DW_TAG_GNU_call_site) {
+ // In DWARF v4 DW_AT_low_pc is always the address of the instruction after
+ // the call. For tail calls, approximate the address of the call
+ // instruction by subtracting 1.
+ if (tail_call)
+ call_inst_pc = low_pc - 1;
+ else
+ return_pc = low_pc;
+ }
+
if (!call_origin && !call_target) {
LLDB_LOG(log, "CollectCallEdges: call site without any call target");
continue;
Index: lldb/source/Expression/DWARFExpression.cpp
===================================================================
--- lldb/source/Expression/DWARFExpression.cpp
+++ lldb/source/Expression/DWARFExpression.cpp
@@ -396,6 +396,7 @@
return offset - data_offset;
}
+ case DW_OP_GNU_entry_value:
case DW_OP_entry_value: // 0xa3 ULEB128 size + variable-length block
{
uint64_t subexpr_len = data.GetULEB128(&offset);
@@ -2522,6 +2523,7 @@
stack.push_back(Scalar(value));
} break;
+ case DW_OP_GNU_entry_value:
case DW_OP_entry_value: {
if (!Evaluate_DW_OP_entry_value(stack, exe_ctx, reg_ctx, opcodes, offset,
error_ptr, log)) {
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
