Skip to content
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

[libcxx][test] Mark sort.pass.cpp as a long test #100720

Merged
merged 1 commit into from
Jul 31, 2024

Conversation

DavidSpickett
Copy link
Collaborator

Picolib testing skips any test requiring this feature, I just didn't know the feature existed until now.

Picolib testing skips any test requiring this feature, I just didn't
know the feature existed until now.
@DavidSpickett DavidSpickett requested a review from a team as a code owner July 26, 2024 09:10
@llvmbot llvmbot added the libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi. label Jul 26, 2024
@llvmbot
Copy link
Collaborator

llvmbot commented Jul 26, 2024

@llvm/pr-subscribers-libcxx

Author: David Spickett (DavidSpickett)

Changes

Picolib testing skips any test requiring this feature, I just didn't know the feature existed until now.


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

1 Files Affected:

  • (modified) libcxx/test/std/algorithms/alg.sorting/alg.sort/sort/sort.pass.cpp (+1-1)
diff --git a/libcxx/test/std/algorithms/alg.sorting/alg.sort/sort/sort.pass.cpp b/libcxx/test/std/algorithms/alg.sorting/alg.sort/sort/sort.pass.cpp
index da7794e8c85dc..9bd28140349b6 100644
--- a/libcxx/test/std/algorithms/alg.sorting/alg.sort/sort/sort.pass.cpp
+++ b/libcxx/test/std/algorithms/alg.sorting/alg.sort/sort/sort.pass.cpp
@@ -8,7 +8,7 @@
 
 // This test did pass but is very slow when run using qemu. ~7 minutes on a
 // Neoverse N1 (AArch64) server core.
-// UNSUPPORTED: LIBCXX-PICOLIBC-FIXME
+// REQUIRES: long_tests
 
 // <algorithm>
 

@DavidSpickett
Copy link
Collaborator Author

It could be !LIBCXX-PICOLIBC-FIXME || long_tests, but no existing test does this. So I'm assuming it's ok to be less specific.

Copy link

@vrukesh vrukesh left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@vrukesh vrukesh left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@vrukesh vrukesh left a comment

Choose a reason for hiding this comment

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

LGTM

@DavidSpickett DavidSpickett merged commit f90e51a into llvm:main Jul 31, 2024
56 checks passed
@DavidSpickett DavidSpickett deleted the libcxx-longtest branch July 31, 2024 14:35
clementval pushed a commit to clementval/llvm-project that referenced this pull request Jul 31, 2024
Picolib testing skips any test requiring this feature, I just didn't
know the feature existed until now.
banach-space pushed a commit to banach-space/llvm-project that referenced this pull request Aug 7, 2024
Picolib testing skips any test requiring this feature, I just didn't
know the feature existed until now.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants