[PATCH] D66254: Correct include suggestion when search path includes symlink

Wed Aug 14 14:13:09 PDT 2019

puremourning added a comment.

In D66254#1630277 <https://reviews.llvm.org/D66254#1630277>, @lebedev.ri wrote:

> Tests?

Per my comment: I haven't added new tests to HeaderSearchTest.cc because the InMemoryFileSystem doesn't support symlinks and I didn't want to try and implement that for this patch. Let me know thoughts on alternative ways to regression test it.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66254