llvmbot wrote:

<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: Nico Weber (nico)

<details>
<summary>Changes</summary>



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


2 Files Affected:

- (modified) clang/include/clang/Options/Options.td (+1-1) 
- (modified) clang/test/Driver/cl-options.c (+2) 


``````````diff
diff --git a/clang/include/clang/Options/Options.td 
b/clang/include/clang/Options/Options.td
index d4dd71b9d1bea..9bf195e3dab4d 100644
--- a/clang/include/clang/Options/Options.td
+++ b/clang/include/clang/Options/Options.td
@@ -3449,7 +3449,7 @@ def fmodules_validate_once_per_build_session : 
Flag<["-"], "fmodules-validate-on
            "successfully validated or loaded during this build session">,
   MarshallingInfoFlag<HeaderSearchOpts<"ModulesValidateOncePerBuildSession">>;
 def fmodules_disable_diagnostic_validation : Flag<["-"], 
"fmodules-disable-diagnostic-validation">,
-  Group<i_Group>, Visibility<[ClangOption, CC1Option]>,
+  Group<i_Group>, Visibility<[ClangOption, CLOption, CC1Option]>,
   HelpText<"Disable validation of the diagnostic options when loading the 
module">,
   
MarshallingInfoNegativeFlag<HeaderSearchOpts<"ModulesValidateDiagnosticOptions">>;
 defm modules_force_validate_user_headers : BoolOption<"f", 
"modules-force-validate-user-headers",
diff --git a/clang/test/Driver/cl-options.c b/clang/test/Driver/cl-options.c
index 1b1169b71554a..8f2f7e4c18020 100644
--- a/clang/test/Driver/cl-options.c
+++ b/clang/test/Driver/cl-options.c
@@ -746,6 +746,8 @@
 // RUN:     -fsystem-module \
 // RUN:     -fmodule-map-file=foo \
 // RUN:     -fmodule-file=foo \
+// RUN:     -fmodules-disable-diagnostic-validation \
+// RUN:     -fno-modules-disable-diagnostic-validation \
 // RUN:     -fmodules-ignore-macro=foo \
 // RUN:     -fmodules-strict-decluse \
 // RUN:     -fmodules-decluse \

``````````

</details>


https://github.com/llvm/llvm-project/pull/176285
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to