<div><div dir="auto">> <span style="color:rgb(49,49,49);word-spacing:1px">I'm also not sure if `-L` is a feature that is relied upon at the moment by anybody that uses the baremetal driver</span></div></div><div dir="auto"><span style="color:rgb(49,49,49);word-spacing:1px"><br></span></div><div dir="auto"><span style="color:rgb(49,49,49);word-spacing:1px">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.</span></div><div dir="auto"><span style="color:rgb(49,49,49);word-spacing:1px"><br></span></div><div dir="auto"><font color="#313131"><span style="word-spacing:1px">re: the -march= stuff: I thought I had added selection logic in the driver for that 🤔. Perhaps that never got upstreamed.</span></font></div><div dir="auto"><font color="#313131"><span style="word-spacing:1px"><br></span></font></div><div dir="auto"><font color="#313131"><span style="word-spacing:1px"><br></span></font></div><div dir="auto"><font color="#313131"><span style="word-spacing:1px">Cheers,</span></font></div><div dir="auto"><font color="#313131"><span style="word-spacing:1px"><br></span></font></div><div dir="auto"><font color="#313131"><span style="word-spacing:1px">Jon</span></font></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Feb 12, 2020 at 4:28 AM Christof Douma via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">christof added a comment.<br>
<br>
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.<br>
<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D73904/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D73904/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D73904" rel="noreferrer" target="_blank">https://reviews.llvm.org/D73904</a><br>
<br>
<br>
<br>
</blockquote></div></div>