Skip to content

Commit 27c5a9b

Browse files
committed
[lldb][test] TestConstStaticIntegralMember.py: fix on older clang versions
`638a8393615e911b729d5662096f60ef49f1c65e` removed the `dsym` condition for older compiler versions which caused the `dwarf` variants tests to XPASS. This patch reverts to only XFAIL-ing the `dsym` variant. `15c80852028ff4020b3f85ee13ad3a2ed4bce3be` added `test_shadowed_static_inline_members` which isn't supported on older compiler versions.
1 parent d9be232 commit 27c5a9b

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

lldb/test/API/lang/cpp/const_static_integral_member/TestConstStaticIntegralMember.py

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,9 @@ def test(self):
105105
# For debug-info produced by older versions of clang, dsymutil strips the
106106
# debug info for classes that only have const static data members without
107107
# definitions.
108-
@expectedFailureAll(compiler=["clang"], compiler_version=["<", "18.0"])
108+
@expectedFailureAll(
109+
debug_info=["dsym"], compiler=["clang"], compiler_version=["<", "18.0"]
110+
)
109111
def test_class_with_only_const_static(self):
110112
self.build()
111113
lldbutil.run_to_source_breakpoint(
@@ -172,6 +174,9 @@ def test_class_with_only_constexpr_static(self):
172174
"ClassWithEnumAlias::enum_alias_alias", result_value="scoped_enum_case1"
173175
)
174176

177+
# With older versions of Clang, LLDB fails to evaluate classes with only
178+
# constexpr members when dsymutil is enabled
179+
@expectedFailureAll(compiler=["clang"], compiler_version=["<", "18.0"])
175180
def test_shadowed_static_inline_members(self):
176181
"""Tests that the expression evaluator and SBAPI can both
177182
correctly determine the requested inline static variable

0 commit comments

Comments
 (0)