[PATCH] D84029: [clang][Driver] Default to /usr/bin/ld on Solaris

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 23 10:24:18 PDT 2020


MaskRay added a comment.

Can you add a test to `test/Driver/solaris-ld.c`?

> However, if someone forgets this, it depends on the user's PATH whether or not clang finds the correct linker, which doesn't make for a good user experience.

Not very sure about this. The last resort of GetProgramPath is `PATH`. On FreeBSD and Linux, this is how `/usr/bin/ld` gets selected. PATH can affect `ld` choice.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84029





More information about the cfe-commits mailing list