hfinkel added a comment.

In https://reviews.llvm.org/D23934#631656, @ed wrote:

> I'd be interested in seeing a feature like this appearing.


I agree.



================
Comment at: lib/Driver/Tools.cpp:4687
+    CmdArgs.push_back(Args.MakeArgString("-ffixed-date-time=" + DateTime));
+  }
+
----------------
This seems like a fairly random place for this code. It is far removed from any 
other handling of preprocessor-related options.


================
Comment at: lib/Driver/Tools.cpp:5996
   }
 
   Args.AddLastArg(CmdArgs, options::OPT_dM);
----------------
Somewhere around here would make more sense.


https://reviews.llvm.org/D23934



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to