Mordante created this revision.
Mordante added a reviewer: ChuanqiXu.
Herald added a project: All.
Mordante requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
Some small style updates as drive-by.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D153038
Files:
clang/include/clang/Basic/DiagnosticSerializationKinds.td
clang/test/Modules/no-implicit-std-cxx-module.cppm
Index: clang/test/Modules/no-implicit-std-cxx-module.cppm
===================================================================
--- clang/test/Modules/no-implicit-std-cxx-module.cppm
+++ clang/test/Modules/no-implicit-std-cxx-module.cppm
@@ -7,7 +7,7 @@
// RUN: -o %t/a.pcm
// RUN: %clang_cc1 -std=c++20 %t/user.cpp -fmodule-file=a=%t/a.pcm -verify
-fsyntax-only
// RUN: %clang_cc1 -std=c++20 %t/user.cpp -fmodule-file=a=%t/a.pcm -verify
-fsyntax-only \
-// RUN: -Wno-read-modules-implicitly -DNO_DIAG
+// RUN: -Wno-read-modules-implicitly -DNO_DIAG
// RUN: %clang_cc1 -std=c++20 %t/user.cpp -fmodule-file=a=%t/a.pcm
-fmodule-file=b=%t/b.pcm \
// RUN: -DNO_DIAG -verify -fsyntax-only
//
@@ -35,7 +35,7 @@
return b() + 43;
}
-// CHECK: it is deprecated to read module 'b' implcitly;
+// CHECK: it is deprecated to read module 'b' implicitly;
// CHECK-CORRECT-NOT: warning
// CHECK-CORRECT-NOT: error
@@ -47,9 +47,9 @@
#ifdef NO_DIAG
// expected-no-diagnostics
#else
- // expected-warning@+2 {{it is deprecated to read module 'b' implcitly;}}
+ // expected-warning@+2 {{it is deprecated to read module 'b' implicitly;}}
#endif
import a;
int use() {
return a();
-}
\ No newline at end of file
+}
Index: clang/include/clang/Basic/DiagnosticSerializationKinds.td
===================================================================
--- clang/include/clang/Basic/DiagnosticSerializationKinds.td
+++ clang/include/clang/Basic/DiagnosticSerializationKinds.td
@@ -135,7 +135,7 @@
InGroup<DiagGroup<"eager-load-cxx-named-modules">>;
def warn_reading_std_cxx_module_by_implicit_paths : Warning<
- "it is deprecated to read module '%0' implcitly; it is going to be removed
in clang18; "
+ "it is deprecated to read module '%0' implicitly; it is going to be removed
in clang 18; "
"consider to specify the dependencies explicitly">,
InGroup<DiagGroup<"read-modules-implicitly">>;
} // let CategoryName
Index: clang/test/Modules/no-implicit-std-cxx-module.cppm
===================================================================
--- clang/test/Modules/no-implicit-std-cxx-module.cppm
+++ clang/test/Modules/no-implicit-std-cxx-module.cppm
@@ -7,7 +7,7 @@
// RUN: -o %t/a.pcm
// RUN: %clang_cc1 -std=c++20 %t/user.cpp -fmodule-file=a=%t/a.pcm -verify -fsyntax-only
// RUN: %clang_cc1 -std=c++20 %t/user.cpp -fmodule-file=a=%t/a.pcm -verify -fsyntax-only \
-// RUN: -Wno-read-modules-implicitly -DNO_DIAG
+// RUN: -Wno-read-modules-implicitly -DNO_DIAG
// RUN: %clang_cc1 -std=c++20 %t/user.cpp -fmodule-file=a=%t/a.pcm -fmodule-file=b=%t/b.pcm \
// RUN: -DNO_DIAG -verify -fsyntax-only
//
@@ -35,7 +35,7 @@
return b() + 43;
}
-// CHECK: it is deprecated to read module 'b' implcitly;
+// CHECK: it is deprecated to read module 'b' implicitly;
// CHECK-CORRECT-NOT: warning
// CHECK-CORRECT-NOT: error
@@ -47,9 +47,9 @@
#ifdef NO_DIAG
// expected-no-diagnostics
#else
- // expected-warning@+2 {{it is deprecated to read module 'b' implcitly;}}
+ // expected-warning@+2 {{it is deprecated to read module 'b' implicitly;}}
#endif
import a;
int use() {
return a();
-}
\ No newline at end of file
+}
Index: clang/include/clang/Basic/DiagnosticSerializationKinds.td
===================================================================
--- clang/include/clang/Basic/DiagnosticSerializationKinds.td
+++ clang/include/clang/Basic/DiagnosticSerializationKinds.td
@@ -135,7 +135,7 @@
InGroup<DiagGroup<"eager-load-cxx-named-modules">>;
def warn_reading_std_cxx_module_by_implicit_paths : Warning<
- "it is deprecated to read module '%0' implcitly; it is going to be removed in clang18; "
+ "it is deprecated to read module '%0' implicitly; it is going to be removed in clang 18; "
"consider to specify the dependencies explicitly">,
InGroup<DiagGroup<"read-modules-implicitly">>;
} // let CategoryName
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
