kbobyrev added a comment.

In D123031#3426303 <https://reviews.llvm.org/D123031#3426303>, @kadircet wrote:

> note that you seem to be using `auto PublicHeader = 
> CanonIncludes.mapHeader(Entry->getName());` (i.e. `getName`) in D120306 
> <https://reviews.llvm.org/D120306> and not `tryGetRealPathName`. it might be 
> nice to have a test case here, at least one that only fails/runs on windows

Right! I think I also need to do that in `SymbolCollector.cpp`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123031

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

Reply via email to