[lld] [LLD] [COFF] Clarify -print-search-path for the empty string element (PR #67856)

Martin Storsjö via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 2 01:33:23 PDT 2023


mstorsjo wrote:

> Looks good, but I wonder about the `(cwd)` - couldn't we print the actual cwd in that case?

I guess we could, although that's a bit messier with respect to tests. (Also, perhaps it kinda loses a little bit of distinction between whether the path actually was specified as an absolute path or not - which probably isn't important?)

In all honesty, it was just the quickest solution I had for making this clearer in tests; I also considered wrapping the path in quotes, which makes it clear that we have one entry with an empty path. But Clang's printing of include directories is a similar feature, which doesn't quote the paths, so I felt it was better to go with consistency with that.


https://github.com/llvm/llvm-project/pull/67856


More information about the llvm-commits mailing list