[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