sammccall added inline comments.

================
Comment at: clang/include/clang/Lex/HeaderSearch.h:710
+  /// Suggest a path by which the specified file could be found, for use in
+  /// diagnostics to suggest a #include. Returned path will be valid for
+  /// include-directive.
----------------
I don't think this is specific enough - on windows, backslashes are valid (or 
at least accepted by clang).
I'd suggest specifically mentioning forward slashes.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60995



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

Reply via email to