kadircet added a comment.

I had just looked at the method name and thought it was the right place( 
`suggestPathToFileForDiagnostics` since it says "path to file".)

But looking at the comments,

  /// Suggest a path by which the specified file could be found, for
  /// use in diagnostics to suggest a #include.

HeaderSearch API itself seems like a better place, I suppose the confusion 
comes from the fact that paths and the include-directives are
the same on unix-like systems.

Updating the patch


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