[PATCH] D73904: [clang] stop baremetal driver to append .a to lib

[Jon Roelofs via cfe-commits]

> I'm also not sure if `-L` is a feature that is relied upon at the moment
by anybody that uses the baremetal driver

Unlikely, IMO. There aren’t that many users of this stuff yet, and it’s
still quite rough around the edges in comparison to the GNU toolchain.
Also, the compiler support libraries really should be bundled with the
compiler, and not dumped in the sysroot or otherwise, which motivates
passing exact paths for this as well.

re: the -march= stuff: I thought I had added selection logic in the driver
for that 🤔. Perhaps that never got upstreamed.


Cheers,

Jon

On Wed, Feb 12, 2020 at 4:28 AM Christof Douma via Phabricator <
reviews at reviews.llvm.org> wrote:

> christof added a comment.
>
> The function @MaskRay suggested does not address the problem with
> `-march=` and others, but it does sound good to use that function. I'm also
> not sure if `-L` is a feature that is relied upon at the moment by anybody
> that uses the baremetal driver. For the moment I'll commit this patch, but
> a single place across drivers that handles the compiler_rt selection
> mechanism certainly sounds nice.
>
>
> CHANGES SINCE LAST ACTION
>   https://reviews.llvm.org/D73904/new/
>
> https://reviews.llvm.org/D73904
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200212/961b4613/attachment.html>