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

[libc++][modules] Fixes naming inconsistency. #83036

Merged
merged 1 commit into from
Feb 27, 2024

Conversation

mordante
Copy link
Member

The modules used is-standard-library and is-std-library. The latter is the name used in the SG15 proposal,

Fixes: #82879

The modules used is-standard-library and is-std-library. The latter is
the name used in the SG15 proposal,

Fixes: llvm#82879
@mordante mordante requested a review from a team as a code owner February 26, 2024 17:18
@llvmbot llvmbot added the libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi. label Feb 26, 2024
@llvmbot
Copy link
Member

llvmbot commented Feb 26, 2024

@llvm/pr-subscribers-libcxx

Author: Mark de Wever (mordante)

Changes

The modules used is-standard-library and is-std-library. The latter is the name used in the SG15 proposal,

Fixes: #82879


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

1 Files Affected:

  • (modified) libcxx/modules/modules.json.in (+1-1)
diff --git a/libcxx/modules/modules.json.in b/libcxx/modules/modules.json.in
index ddc377f28f9194..759ac92d81f18e 100644
--- a/libcxx/modules/modules.json.in
+++ b/libcxx/modules/modules.json.in
@@ -5,7 +5,7 @@
     {
       "logical-name": "std",
       "source-path": "@LIBCXX_MODULE_RELATIVE_PATH@/std.cppm",
-      "is-standard-library": true,
+      "is-std-library": true,
       "local-arguments": {
         "system-include-directories": [
           "@LIBCXX_MODULE_RELATIVE_PATH@"

@ldionne ldionne added this to the LLVM 18.X Release milestone Feb 27, 2024
@ldionne ldionne merged commit b50bcc7 into llvm:main Feb 27, 2024
12 of 14 checks passed
@ldionne
Copy link
Member

ldionne commented Feb 27, 2024

/cherry-pick b50bcc7

llvmbot pushed a commit to llvmbot/llvm-project that referenced this pull request Feb 27, 2024
The modules used is-standard-library and is-std-library. The latter is
the name used in the SG15 proposal,

Fixes: llvm#82879
(cherry picked from commit b50bcc7)
@llvmbot
Copy link
Member

llvmbot commented Feb 27, 2024

/pull-request #83156

@mordante mordante deleted the review/fixes_modules_json_inconsistency branch March 3, 2024 12:23
llvmbot pushed a commit to llvmbot/llvm-project that referenced this pull request Mar 11, 2024
The modules used is-standard-library and is-std-library. The latter is
the name used in the SG15 proposal,

Fixes: llvm#82879
(cherry picked from commit b50bcc7)
@pointhex pointhex mentioned this pull request May 7, 2024
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
Development

Successfully merging this pull request may close these issues.

modules;json: inconsistent keys: is-standard-library vs is-std-library
4 participants