[PATCH] D39104: Allow /showIncludes with /P

2017-10-20 Thread Erich Keane via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL316225: Allow /showIncludes with /P (authored by erichkeane). Changed prior to commit: https://reviews.llvm.org/D39104?vs=119607&id=119686#toc Repository: rL LLVM https://reviews.llvm.org/D39104 Fil

[PATCH] D39104: Allow /showIncludes with /P

2017-10-20 Thread Nico Weber via Phabricator via cfe-commits
thakis accepted this revision. thakis added a comment. This revision is now accepted and ready to land. Thanks! https://reviews.llvm.org/D39104 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cf

[PATCH] D39104: Allow /showIncludes with /P

2017-10-19 Thread Erich Keane via Phabricator via cfe-commits
erichkeane created this revision. r213589 was checked in as a solution to https://bugs.llvm.org/show_bug.cgi?id=20336. However, it is possible to use /EP with /P to suppress #line directives AND output to a file. There is no reason in that case to suppress /showIncludes. This was reported here: