phosek added a comment.

In D110128#3020468 <https://reviews.llvm.org/D110128#3020468>, @nemanjai wrote:

> In D110128#3018992 <https://reviews.llvm.org/D110128#3018992>, @phosek wrote:
>
>> @MaskRay Do you think we should gate the use of this feature on 
>> `-fbinutils-version=` or `-fuse-ld=lld`? It'd be nice if the owner of 
>> `clang-ppc64le-rhel` builder could update the binutils version but I'm not 
>> sure how feasible is it.
>
> We certainly don't mind updating binutils to a supported version. However, if 
> a specific version of binutils (or really any other software package) is 
> required by Clang/LLVM, that should be documented at 
> https://llvm.org/docs/GettingStarted.html#software
>
> Of course, if our bot has a version of binutils that is older than the one 
> listed there currently, we will be sure to update it ASAP.
>
> But of course, as @MaskRay pointed out, this may not be an issue.

Thanks for reaching out @nemanjai, I assume you're responsible for this 
builder? Do you know what version of Binutils is currently installed on that 
machine? Is newer version available for RHEL 7?

We don't specify Binutils version on https://llvm.org/docs/GettingStarted.html 
but I think we should. We need to determine that baseline first though. It 
looks like 2.25 roughly corresponds to GCC 5.1 which is the minimum requirement 
for the compiler.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110128

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

Reply via email to