<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, May 7, 2018 at 11:56 AM, Amaan Cheval <span dir="ltr"><<a href="mailto:amaan.cheval@gmail.com" target="_blank">amaan.cheval@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks a ton! I'll likely make the corresponding RSB patch to backport<br>
this to GCC 7.3 tomorrow for Chris to review :)<br></blockquote><div><br></div><div>If need be, we can address pushing it to open gcc branches. I didn't see it</div><div>as critical as this was a new port but am happy to do it. </div><div><br></div><div>If it goes on the 7 branch, it needs to go on the 8 branch though.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
On Mon, May 7, 2018 at 10:03 PM, Joel Sherrill <<a href="mailto:joel@rtems.org">joel@rtems.org</a>> wrote:<br>
><br>
><br>
> On Fri, May 4, 2018 at 4:25 PM, Joel Sherrill <<a href="mailto:joel@rtems.org">joel@rtems.org</a>> wrote:<br>
>><br>
>><br>
>><br>
>> On Fri, May 4, 2018 at 4:58 AM, Amaan Cheval <<a href="mailto:amaan.cheval@gmail.com">amaan.cheval@gmail.com</a>><br>
>> wrote:<br>
>>><br>
>>> On Fri, May 4, 2018 at 7:54 AM Joel Sherrill <<a href="mailto:joel@rtems.org">joel@rtems.org</a>> wrote:<br>
>>><br>
>>> > I am now confused. Do I need to get a patch into GCC?<br>
>>><br>
>>> Hey!<br>
>>><br>
>>> Sorry about the confusion. Technically the answer is yes, but it's<br>
>>> unrelated to _this_ patch.<br>
>>><br>
>>> To clear things up:<br>
>>><br>
>>> - [DONE] You already committed a previous patch for GCC for -qrtems,<br>
>>> -qnolinkcmds, etc.<br>
>>><br>
>>><br>
>>> <a href="https://gcc.gnu.org/git/?p=gcc.git;a=commitdiff;h=602fa1e9d3ea5e87d4d6e17e3e91fc2647e42da3" rel="noreferrer" target="_blank">https://gcc.gnu.org/git/?p=<wbr>gcc.git;a=commitdiff;h=<wbr>602fa1e9d3ea5e87d4d6e17e3e91fc<wbr>2647e42da3</a><br>
>>><br>
>>> - [DONE] This email thread backports the above patch to<br>
>>> rtems-source-builder so that we can use it in GCC 7.3. Chris accepted<br>
>>> this<br>
>>> patch into rtems-source-builder here<br>
>>><br>
>>><br>
>>> <a href="https://git.rtems.org/rtems-source-builder/commit/?id=defa958301215995b0fa41d8e65cb23c9a28a847" rel="noreferrer" target="_blank">https://git.rtems.org/rtems-<wbr>source-builder/commit/?id=<wbr>defa958301215995b0fa41d8e65cb2<wbr>3c9a28a847</a><br>
>>><br>
>>> - [PENDING - Joel] I recently made a GCC patch to have GCC build crti.o<br>
>>> and<br>
>>> crtn.o for the x86-64 RTEMS tools, so we can have symbols like "_fini"<br>
>>> added automatically through our bsp_specs file, the way we do for other<br>
>>> architectures. As Sebastian mentioned in an email to you and me, I<br>
>>> believe<br>
>>> he'd prefer if you reviewed the patch here:<br>
>>>    <a href="https://gcc.gnu.org/ml/gcc-patches/2018-05/msg00126.html" rel="noreferrer" target="_blank">https://gcc.gnu.org/ml/gcc-<wbr>patches/2018-05/msg00126.html</a><br>
>>><br>
>>> - [PENDING - Amaan, and later Chris] Once the crti.o patch above is<br>
>>> merged<br>
>>> into GCC, I'll make a corresponding backport patch to<br>
>>> rtems-source-builder<br>
>>> to make that available to GCC 7.3 as well, and then Chris can review +<br>
>>> merge that in after.<br>
>>><br>
>> I meant to deal with this today.<br>
>><br>
>> I am leaving myself a note to get this merged next week. Ping me in the<br>
>> middle<br>
>> of next week if you haven't heard action.<br>
><br>
><br>
> This is now committed on the gcc master.<br>
><br>
> <a href="https://gcc.gnu.org/ml/gcc-cvs/2018-05/msg00226.html" rel="noreferrer" target="_blank">https://gcc.gnu.org/ml/gcc-<wbr>cvs/2018-05/msg00226.html</a><br>
><br>
> Hopefully that helps clean up the RSB.<br>
><br>
>><br>
>><br>
>>><br>
>>> Hope that clears things up!<br>
>><br>
>><br>
>> It does.<br>
>>><br>
>>><br>
>>> > On Thu, May 3, 2018, 8:38 PM Chris Johns <<a href="mailto:chrisj@rtems.org">chrisj@rtems.org</a>> wrote:<br>
>>><br>
>>> >> On 07/04/2018 08:35, Amaan Cheval wrote:<br>
>>> >> > Adds -qnolinkcmds, -qrtems, and defaults to pass to linker, for<br>
>>> >> > gcc targeting x86_64.<br>
>>><br>
>>> >> Pushed, thank you.<br>
>>><br>
>>> >> Chris<br>
>>> >> ______________________________<wbr>_________________<br>
>>> >> devel mailing list<br>
>>> >> <a href="mailto:devel@rtems.org">devel@rtems.org</a><br>
>>> >> <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/<wbr>mailman/listinfo/devel</a><br>
>>><br>
>>> > ______________________________<wbr>_________________<br>
>>> > devel mailing list<br>
>>> > <a href="mailto:devel@rtems.org">devel@rtems.org</a><br>
>>> > <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/<wbr>mailman/listinfo/devel</a><br>
>><br>
>><br>
><br>
</div></div></blockquote></div><br></div></div>