================
@@ -574,18 +606,31 @@ HLSLToolChain::TranslateArgs(const DerivedArgList &Args,
StringRef BoundArch,
return DAL;
}
-bool HLSLToolChain::requiresValidation(DerivedArgList &Args) const {
- if (!Args.hasArg(options::OPT_dxc_Fo))
+bool HLSLToolChain::requiresValidation(DerivedArgList &Args,
+ bool Diagnose) const {
----------------
bogner wrote:
I guess the addition of the `Diagnose` argument here fixes an existing bug
where we double diagnose if dxv is missing?
https://github.com/llvm/llvm-project/pull/188150
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
