<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, May 7, 2018 at 12:16 PM, 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">I'm not quite sure how GCC's releases work; it seems like GCC 8.1 has<br>
already been released:<br>
<a href="https://gcc.gnu.org/releases.html" rel="noreferrer" target="_blank">https://gcc.gnu.org/releases.<wbr>html</a><br>
<a href="https://ftp.gnu.org/gnu/gcc/gcc-8.1.0/" rel="noreferrer" target="_blank">https://ftp.gnu.org/gnu/gcc/<wbr>gcc-8.1.0/</a><br>
<br>
However, the "gcc-7-branch" is still being pushed to:<br>
<a href="https://gcc.gnu.org/git/?p=gcc.git;a=shortlog;h=refs/heads/gcc-7-branch" rel="noreferrer" target="_blank">https://gcc.gnu.org/git/?p=<wbr>gcc.git;a=shortlog;h=refs/<wbr>heads/gcc-7-branch</a><br>
<br>
What would pushing to that branch entail? Is it simply to make<br>
applying the patch to a 7.3 release easier?<br>
<br>
I only meant to make the patch to the RSB because:<br>
- I'm not sure when the RSB will move to using GCC 8.1+ (we still use 7.3)<br>
- It's better to have the RSB build the tooling the same for my dev<br>
environment as the community will have, without needing manual patches<br>
for building the tools<br>
<br>
I'm okay to just make a note about the patch if we'd rather not have<br>
it in the RSB in case by the end of the port, GCC has released a<br>
version including the patches which we can simply switch to using.<br>
<br>
Let me know if you prefer the latter!<br></blockquote><div><br></div><div>Unless gcc makes a release from the branch, we will have to continue</div><div>to have a patch. </div><div><br></div><div>Chris is on holiday. Let's see what he wants done when he gets back.</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:28 PM, Joel Sherrill <<a href="mailto:joel@rtems.org">joel@rtems.org</a>> wrote:<br>
><br>
><br>
> On Mon, May 7, 2018 at 11:56 AM, Amaan Cheval <<a href="mailto:amaan.cheval@gmail.com">amaan.cheval@gmail.com</a>><br>
> wrote:<br>
>><br>
>> 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>
><br>
><br>
> If need be, we can address pushing it to open gcc branches. I didn't see it<br>
> as critical as this was a new port but am happy to do it.<br>
><br>
> If it goes on the 7 branch, it needs to go on the 8 branch though.<br>
><br>
>><br>
>><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>
>> >>><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>
>> >>><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<br>
>> >>> 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<br>
>> >>> 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>
><br>
><br>
</div></div></blockquote></div><br></div></div>