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

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

Reply via email to