[PATCH] D122110: [Clang] Set -mlinker-version explicitly in the lto test

Mon Mar 21 11:27:41 PDT 2022

phosek added a comment.

In D122110#3395427 <https://reviews.llvm.org/D122110#3395427>, @int3 wrote:

> Ohh gotcha. So clang assumes that the platform linker's version is at least as high as the host linker's, assuming the host linker is ld64. That seems kind of hacky, but I suppose it's existing behavior...

I agree, I don't think that the version of the host linker on the system where Clang was built should affect driver's behavior. Ideally, Clang would check the linker version dynamically but that would be a change to the existing behavior and I don't know if that would cause problems.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122110