-
Notifications
You must be signed in to change notification settings - Fork 13.5k
[libc++] Properly define _LIBCPP_HAS_NO_UNICODE in __config_site #95138
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
@llvm/pr-subscribers-libcxx Author: Louis Dionne (ldionne) ChangesFixes #93638 Full diff: https://github.com/llvm/llvm-project/pull/95138.diff 1 Files Affected:
diff --git a/libcxx/include/__config_site.in b/libcxx/include/__config_site.in
index 89a14609ee3f9..67022146c9082 100644
--- a/libcxx/include/__config_site.in
+++ b/libcxx/include/__config_site.in
@@ -26,6 +26,7 @@
#cmakedefine _LIBCPP_HAS_NO_FILESYSTEM
#cmakedefine _LIBCPP_HAS_NO_RANDOM_DEVICE
#cmakedefine _LIBCPP_HAS_NO_LOCALIZATION
+#cmakedefine _LIBCPP_HAS_NO_UNICODE
#cmakedefine _LIBCPP_HAS_NO_WIDE_CHARACTERS
#cmakedefine _LIBCPP_HAS_NO_STD_MODULES
#cmakedefine _LIBCPP_HAS_NO_TIME_ZONE_DATABASE
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! LGTM!
@mordante , I am getting CI failures in
However, I am really not certain how to best fix those in most cases. Since this is more your cup of tea, do you think you could take a look? I suspect you'll know what the right fixes are right away. |
From a quick look this looks like broken tests due to |
libcxx/test/libcxx/utilities/format/format.functions/ascii.pass.cpp
Outdated
Show resolved
Hide resolved
Two test still fail but need a bit more investigation.
f3438ad
to
b45a166
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still LGTM!
…m#95138) Fixes llvm#93638 Co-authored-by: Mark de Wever <koraq@xs4all.nl>
Fixes #93638