[PATCH] D155047: [lld][COFF] Add -print-search-paths for debugging.

Martin Storsjö via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 12 13:06:55 PDT 2023


mstorsjo added a comment.

No objections from me, looks reasonable.

Just make sure the test passes on both unix and Windows - which I believe the premerge CI should cover here. There's always a bit of fiddliness regarding absolute paths and literally checking for matches; e.g. if the tool would happen to renormalize path separator style within that section, that would break. But I believe we use that pattern in a few other places as well so it's most probably fine.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155047



More information about the llvm-commits mailing list