================
@@ -3854,6 +3854,25 @@ static void RenderHLSLOptions(const Driver &D, const 
ArgList &Args,
   }
   if (Arg *A = Args.getLastArg(options::OPT_dxc_Zsb))
     A->claim(); // /Zsb is the default behavior, no need to forward it to llc.
+  bool Zi = Args.hasArg(options::OPT_g_Flag);
----------------
bogner wrote:

Zi is a confusing name for this variable since it's checking `-g`. It'd 
probably be better to call it `HasDebug` or something to that effect. This also 
means our diagnostics (which match dxc) being specific about `/Zi` are a bit 
misleading, since `clang-dxc` also accepts `-g`, but that's probably not a big 
deal.

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

Reply via email to