[PATCH] D156781: Use the canonical path for the include header search.

Jonas via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 1 12:39:40 PDT 2023


felberj added a comment.

Thanks a lot for your notes! Unfortunately I noticed that while the test that I created passed, my actual issue [0] still persists and I sent out the change too early. It seems you are right and the symlink handling is in fact more nuanced.

Enjoy your vacation! I will go on vacation soon as well, so no hurries with the review.

[0] https://github.com/hedronvision/bazel-compile-commands-extractor/issues/136


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156781



More information about the cfe-commits mailing list