ormris added inline comments.

================
Comment at: test/ClangScanDeps/regular_cdb.cpp:14
+// CHECK: regular_cdb.cpp
+// CHECK-NEXT: Inputs/header.h
----------------
Quick drive-by nit: This test won't work under Windows due to the path 
separator. To account for both platforms, you could use a pattern like 
"{{/|\\}}".


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60233/new/

https://reviews.llvm.org/D60233



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

Reply via email to