Skip to content

[lldb][Windows] Skip the TestDataFormatterLibcxxChrono test to avoid python crash #92575

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

Conversation

slydiman
Copy link
Contributor

The python crashed with the exit code 0xC0000409 (STATUS_STACK_BUFFER_OVERRUN) on the command frame variable ss_neg_seconds running on Windows x86_64. See this issue for details #92574

…python crash

The python crashed with the exit code 0xC0000409 (STATUS_STACK_BUFFER_OVERRUN) on the command `frame variable ss_neg_seconds` running on Windows x86_64. See this issue for details llvm#92574
@slydiman slydiman requested a review from JDevlieghere as a code owner May 17, 2024 16:47
@slydiman slydiman requested a review from labath May 17, 2024 16:47
@llvmbot llvmbot added the lldb label May 17, 2024
@slydiman slydiman requested a review from DavidSpickett May 17, 2024 16:47
@llvmbot
Copy link
Member

llvmbot commented May 17, 2024

@llvm/pr-subscribers-lldb

Author: Dmitry Vasilyev (slydiman)

Changes

The python crashed with the exit code 0xC0000409 (STATUS_STACK_BUFFER_OVERRUN) on the command frame variable ss_neg_seconds running on Windows x86_64. See this issue for details #92574


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

1 Files Affected:

  • (modified) lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx/chrono/TestDataFormatterLibcxxChrono.py (+3)
diff --git a/lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx/chrono/TestDataFormatterLibcxxChrono.py b/lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx/chrono/TestDataFormatterLibcxxChrono.py
index fb35481d55514..bb7310bc1a8c1 100644
--- a/lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx/chrono/TestDataFormatterLibcxxChrono.py
+++ b/lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx/chrono/TestDataFormatterLibcxxChrono.py
@@ -12,6 +12,9 @@
 class LibcxxChronoDataFormatterTestCase(TestBase):
     @add_test_categories(["libc++"])
     @skipIf(compiler="clang", compiler_version=["<", "17.0"])
+    @skipIf(
+        hostoslist=["windows"], bugnumber="github.com/llvm/llvm-project/issues/92574"
+    )
     def test_with_run_command(self):
         """Test that that file and class static variables display correctly."""
         self.build()

@Michael137
Copy link
Member

Michael137 commented May 17, 2024

I think we should just fix #92574 since it seems pretty trivial to do with an ifdef, and this seems like a serious issue when debugging on Windows

@slydiman
Copy link
Contributor Author

@Michael137

I think we should just fix #92574 since it seems pretty trivial

Please look at #92701.

slydiman added a commit to slydiman/llvm-project that referenced this pull request May 19, 2024
slydiman added a commit that referenced this pull request May 20, 2024
@slydiman slydiman closed this May 20, 2024
@slydiman slydiman deleted the fix-lldb-test-TestDataFormatterLibcxxChrono branch July 25, 2024 21:30
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.

3 participants