[PATCH] D69763: [Clang][Test]: Remaining "lld-link2" -> "lld-link"

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 21 09:52:00 PDT 2023


MaskRay added a comment.

> So this is truly a test of the ld.otherlinker feature pattern, not some special case driver feature. I guess we should leave the test alone.
> Closing, we left the test alone, it still uses -fuse-ld=lld-link2. Perhaps in the future we should reconsider this, but that's how things stand now, and we aren't going to land this patch as is. + @MaskRay , who has thought about -fuse-ld= semantics.

Yes, we can leave the test alone. The `ld.<name>` convention is mainly for ELF.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69763



More information about the cfe-commits mailing list