Skip to content

Commit 6385b2c

Browse files
committed
Don't warn
1 parent 27ce98f commit 6385b2c

File tree

3 files changed

+0
-22
lines changed

3 files changed

+0
-22
lines changed

clang/include/clang/Basic/DiagnosticDriverKinds.td

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -555,13 +555,6 @@ def err_test_module_file_extension_format : Error<
555555
def err_drv_module_output_with_multiple_arch : Error<
556556
"option '-fmodule-output' cannot be used with multiple arch options">;
557557

558-
def warn_drv_module_reduced_bmi_not_enabled : Warning<
559-
"reduced BMI is expected to be enabled by default in Clang 21; it can be enabled "
560-
"explicitly by passing '-fmodules-reduced-bmi' (in one-phase compilation models, "
561-
"like CMake) or by disabling the diagnostic and requesting the build system vendor "
562-
"support reduced BMIs (in two-phase compilation models)">,
563-
InGroup<DiagGroup<"missing-reduced-bmi">>;
564-
565558
def warn_drv_delayed_template_parsing_after_cxx20 : Warning<
566559
"-fdelayed-template-parsing is deprecated after C++20">,
567560
InGroup<DiagGroup<"delayed-template-parsing-in-cxx20">>;

clang/lib/Driver/ToolChains/Clang.cpp

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -4267,14 +4267,6 @@ static bool RenderModulesOptions(Compilation &C, const Driver &D,
42674267
getCXX20NamedModuleOutputPath(Args, Input.getBaseInput())));
42684268
}
42694269

4270-
if ((Input.getType() == driver::types::TY_CXXModule ||
4271-
Input.getType() == driver::types::TY_PP_CXXModule) &&
4272-
(Args.hasArg(options::OPT_fmodule_output) ||
4273-
Args.hasArg(options::OPT_fmodule_output_EQ) ||
4274-
Args.hasArg(options::OPT__precompile)) &&
4275-
!Args.hasArg(options::OPT_modules_reduced_bmi))
4276-
D.Diag(diag::warn_drv_module_reduced_bmi_not_enabled);
4277-
42784270
// Noop if we see '-fmodules-reduced-bmi' with other translation
42794271
// units than module units. This is more user friendly to allow end uers to
42804272
// enable this feature without asking for help from build systems.

clang/test/Driver/module-fgen-reduced-bmi.cppm

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -53,12 +53,6 @@
5353
// RUN: -c -o Hello.o -### 2>&1 | FileCheck Hello.cc
5454
//
5555
// RUN: %clang -std=c++20 Hello.cppm -fmodule-output=Hello.pcm -c -o Hello.o \
56-
// RUN: -### 2>&1 | FileCheck Hello.cppm -check-prefix=WARN
57-
//
58-
// RUN: %clang -std=c++20 Hello.cppm --precompile -o Hello.pcm \
59-
// RUN: -### 2>&1 | FileCheck Hello.cppm -check-prefix=WARN
60-
//
61-
// RUN: %clang -std=c++20 Hello.cppm -fmodule-output=Hello.pcm -c -o Hello.o \
6256
// RUN: -Wno-missing-reduced-bmi -### 2>&1 | FileCheck Hello.cppm -check-prefix=NO_WARN
6357
//
6458
// RUN: %clang -std=c++20 Hello.cppm --precompile -o Hello.pcm \
@@ -80,7 +74,6 @@ export module Hello;
8074
// flag.
8175
// CHECK-EMIT-MODULE-INTERFACE: -emit-module-interface
8276

83-
// WARN: reduced BMI is expected to be enabled by default in Clang 21;
8477
// NO_WARN-NOT: warning
8578

8679
//--- Hello.cc

0 commit comments

Comments
 (0)