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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to