[PATCH] D30743: enable -save-temps with -finclude-defult-header

Anastasia Stulova via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 14 07:08:16 PDT 2017


Anastasia accepted this revision.
Anastasia added a comment.
This revision is now accepted and ready to land.

LGTM! Thanks!



================
Comment at: lib/Driver/Tools.cpp:5290
+      !isa<PreprocessJobAction>(JA)) {
+    //Args.AddAllArgValues(CmdArgs, options::OPT_Xclang);
+    for (auto Arg : Args.filtered(options::OPT_Xclang)) {
----------------
Anastasia wrote:
> Remove the commented code, please!
I think the comment could be lifted up before the if statement. :)


https://reviews.llvm.org/D30743





More information about the cfe-commits mailing list