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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to