[PATCH] D53249: Force Hexagon to use default (hexagon-link) linker

Petr Hosek via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 15 18:13:09 PDT 2018


phosek added a comment.

In https://reviews.llvm.org/D53249#1264900, @sidneym wrote:

> https://reviews.llvm.org/D53219 added a check for either hexagon-ld or ld.  I merged that change earlier today.  I can change the test to pass -fuse-ld, let me know.  Thanks,


Passing `-fuse-ld=ld` is less error prone than checking for `hexagon-link` or `*ld*` in case someone sets the default linker to something else, e.g. `link.exe` on Windows or something like `foo-link` on some new platform.


Repository:
  rC Clang

https://reviews.llvm.org/D53249





More information about the cfe-commits mailing list