[PATCH] D110128: [Driver] Correctly handle static C++ standard library

Fri Sep 24 10:37:29 PDT 2021

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