Skip to content

[lldb-dap] Test: disable children return test for all arm architectures #129409

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 4, 2025

Conversation

da-viper
Copy link
Contributor

@da-viper da-viper commented Mar 2, 2025

amd64 and aarch64 are treated differently

Follows up #106907

@da-viper da-viper requested a review from JDevlieghere as a code owner March 2, 2025 01:18
@llvmbot llvmbot added the lldb label Mar 2, 2025
@llvmbot
Copy link
Member

llvmbot commented Mar 2, 2025

@llvm/pr-subscribers-lldb

Author: None (Da-Viper)

Changes

amd64 and aarch64 are treated differently

Follows up #106907


Full diff: https://github.com/llvm/llvm-project/pull/129409.diff

1 Files Affected:

  • (modified) lldb/test/API/tools/lldb-dap/variables/children/TestDAP_variables_children.py (+1-1)
diff --git a/lldb/test/API/tools/lldb-dap/variables/children/TestDAP_variables_children.py b/lldb/test/API/tools/lldb-dap/variables/children/TestDAP_variables_children.py
index f0e1e51365d9b..a9371e5c5fe68 100644
--- a/lldb/test/API/tools/lldb-dap/variables/children/TestDAP_variables_children.py
+++ b/lldb/test/API/tools/lldb-dap/variables/children/TestDAP_variables_children.py
@@ -41,7 +41,7 @@ def test_get_num_children(self):
             )["body"]["result"],
         )
 
-    @skipIf(archs=["arm64"])
+    @skipIf(archs=["arm", "arm64", "aarch64"])
     def test_return_variable_with_children(self):
         """
         Test the stepping out of a function with return value show the children correctly

@@ -41,7 +41,7 @@ def test_get_num_children(self):
)["body"]["result"],
)

@skipIf(archs=["arm64"])
@skipIf(archs=["arm", "arm64", "aarch64"])
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what about arm64e? Does this test pass on arm64e?

@omjavaid omjavaid self-requested a review March 4, 2025 10:39
@omjavaid omjavaid merged commit c7dbf20 into llvm:main Mar 4, 2025
12 checks passed
@omjavaid
Copy link
Contributor

omjavaid commented Mar 4, 2025

@da-viper I am sorry but this PR was required to make LLDB Windows buildbot green so I have taken the liberty to merge it on your behalf.

@da-viper da-viper deleted the show_return_value_on_step_out branch March 16, 2025 03:24
jph-13 pushed a commit to jph-13/llvm-project that referenced this pull request Mar 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants