================
@@ -7344,6 +7346,28 @@ void Clang::ConstructJob(Compilation &C, const JobAction 
&JA,
     CmdArgs.push_back(Args.MakeArgString("-ftime-trace=" + Twine(Name)));
     Args.AddLastArg(CmdArgs, options::OPT_ftime_trace_granularity_EQ);
     Args.AddLastArg(CmdArgs, options::OPT_ftime_trace_verbose);
+    if (const Arg *CompressArg =
+            Args.getLastArg(options::OPT_ftime_trace_compress_EQ)) {
+      StringRef Value = CompressArg->getValue();
+      if (Value == "none") {
+        // Explicit -ftime-trace-compress=none disables compression even if the
+        // filename has a .zst/.zstd extension.
----------------
alinas wrote:

Nit: does it make sense to have a warning diagnostic when inferPath returns 
zstd?
I'm not clear what would be a use case for the mismatch -- when the path 
argument cannot be updated and needs to be overwritten by 
`-ftime-trace-compress=none`?

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

Reply via email to