Skip to content

Conversation

hawkinsw
Copy link
Contributor

After changes to , more test files are now properly formatted.

After changes to <expected>, more test files are now properly formatted.
@hawkinsw hawkinsw self-assigned this Oct 31, 2023
@hawkinsw hawkinsw requested a review from a team as a code owner October 31, 2023 22:38
@llvmbot llvmbot added the libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi. label Oct 31, 2023
@llvmbot
Copy link
Member

llvmbot commented Oct 31, 2023

@llvm/pr-subscribers-libcxx

Author: Will Hawkins (hawkinsw)

Changes

After changes to <expected>, more test files are now properly formatted.


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

1 Files Affected:

  • (modified) libcxx/utils/data/ignore_format.txt (-2)
diff --git a/libcxx/utils/data/ignore_format.txt b/libcxx/utils/data/ignore_format.txt
index d4afd8c87350fd1..cf6dca7cd7d3c68 100644
--- a/libcxx/utils/data/ignore_format.txt
+++ b/libcxx/utils/data/ignore_format.txt
@@ -6257,7 +6257,6 @@ libcxx/test/std/utilities/expected/expected.expected/observers/deref.pass.cpp
 libcxx/test/std/utilities/expected/expected.expected/observers/error.pass.cpp
 libcxx/test/std/utilities/expected/expected.expected/observers/has_value.pass.cpp
 libcxx/test/std/utilities/expected/expected.expected/observers/value.pass.cpp
-libcxx/test/std/utilities/expected/expected.expected/swap/member.swap.pass.cpp
 libcxx/test/std/utilities/expected/expected.unexpected/ctad.compile.pass.cpp
 libcxx/test/std/utilities/expected/expected.unexpected/ctor/ctor.inplace.pass.cpp
 libcxx/test/std/utilities/expected/expected.unexpected/ctor/ctor.move.pass.cpp
@@ -6277,7 +6276,6 @@ libcxx/test/std/utilities/expected/expected.void/observers/deref.pass.cpp
 libcxx/test/std/utilities/expected/expected.void/observers/error.pass.cpp
 libcxx/test/std/utilities/expected/expected.void/observers/has_value.pass.cpp
 libcxx/test/std/utilities/expected/expected.void/swap/free.swap.pass.cpp
-libcxx/test/std/utilities/expected/expected.void/swap/member.swap.pass.cpp
 libcxx/test/std/utilities/format/format.arguments/format.args/get.pass.cpp
 libcxx/test/std/utilities/format/format.arguments/format.args/types.compile.pass.cpp
 libcxx/test/std/utilities/format/format.arguments/format.arg/visit_format_arg.pass.cpp

@hawkinsw hawkinsw requested a review from huixie90 October 31, 2023 22:41
Copy link
Contributor

@philnik777 philnik777 left a comment

Choose a reason for hiding this comment

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

Thanks!

@hawkinsw
Copy link
Contributor Author

Thanks!

My pleasure. I will land as soon as the checks complete, if that's okay!

@philnik777
Copy link
Contributor

I'm landing this now, since the relevant tests passed and this is breaking everybody's CI pipeline.

@philnik777 philnik777 merged commit 8c0e296 into llvm:main Oct 31, 2023
@hawkinsw
Copy link
Contributor Author

Thanks!! I didn't want to be presumptuous and jump the gun!

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.

3 participants