[PATCH] D25153: preprocessor supports `-dI` flag

Steve O'Brien via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 28 10:50:16 PDT 2016


elsteveogrande added inline comments.


================
Comment at: cfe/trunk/test/Preprocessor/dump_include.c:2
+// RUN: %clang_cc1 -w -E -dI -isystem %S -imacros %S/dump_include.h %s -o - | FileCheck %s
+// CHECK: {{^}}#__include_macros "/{{([^/]+/)+}}dump_
+// CHECK: {{^}}#include <dump_
----------------
Thanks very much @bruno and sorry for the trouble here!  Looks like I was expecting a forward slash at the beginning, forgetting about drive letters on Windows -- and backslashes (actually looks like backslashes and forward slashes can get mixed?).

Indeed the output on windows looked like:

  #__include_macros "C:\b\slave\clang-x86-windows-msvc2015\clang-x86-windows-msvc2015\llvm\tools\clang\test\Preprocessor/dump_include.h" /* clang -E -dI */

which actually looks correct.  I'll have to correct my test here for this case.  Will have this updated in a couple of minutes.  Thanks very much!


Repository:
  rL LLVM

https://reviews.llvm.org/D25153





More information about the cfe-commits mailing list