[PATCH] D46394: [clang-cl] Print /showIncludes to stderr, if used in combination with /E, /EP or /P

[Felix Bruns via Phabricator via cfe-commits]

fxb updated this revision to Diff 145196.
fxb added a comment.

1. Fixed the compile error caused by re-using the name `ShowIncludesDestination`. The member variable is now named `ShowIncludesDest`.
2. Fixed `test/Frontend/print-header-includes.c` to test both cases:
  - If only `--show-includes` is passed, includes are printed on stdout.
  - If  both `--show-includes` and `-E` are passed, includes are printed on stderr.




https://reviews.llvm.org/D46394

Files:
  include/clang/Frontend/DependencyOutputOptions.h
  lib/Driver/ToolChains/Clang.cpp
  lib/Frontend/CompilerInstance.cpp
  lib/Frontend/CompilerInvocation.cpp
  lib/Frontend/HeaderIncludeGen.cpp
  test/Driver/cl-options.c
  test/Frontend/print-header-includes.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D46394.145196.patch
Type: text/x-patch
Size: 8072 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180504/c55925dc/attachment.bin>