[PATCH] D123031: [clangd] Use consistent header paths in CanonicalIncludes mappings

Kadir Cetinkaya via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 4 07:34:00 PDT 2022


kadircet added a comment.

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


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123031



More information about the cfe-commits mailing list