[PATCH v2] x86_64/gcc: Add gcc patch for LIB_SPEC (rtems.h in gcc)
Amaan Cheval
amaan.cheval at gmail.com
Mon May 7 16:56:23 UTC 2018
Thanks a ton! I'll likely make the corresponding RSB patch to backport
this to GCC 7.3 tomorrow for Chris to review :)
On Mon, May 7, 2018 at 10:03 PM, Joel Sherrill <joel at rtems.org> wrote:
>
>
> On Fri, May 4, 2018 at 4:25 PM, Joel Sherrill <joel at rtems.org> wrote:
>>
>>
>>
>> On Fri, May 4, 2018 at 4:58 AM, Amaan Cheval <amaan.cheval at gmail.com>
>> wrote:
>>>
>>> On Fri, May 4, 2018 at 7:54 AM Joel Sherrill <joel at rtems.org> wrote:
>>>
>>> > I am now confused. Do I need to get a patch into GCC?
>>>
>>> Hey!
>>>
>>> Sorry about the confusion. Technically the answer is yes, but it's
>>> unrelated to _this_ patch.
>>>
>>> To clear things up:
>>>
>>> - [DONE] You already committed a previous patch for GCC for -qrtems,
>>> -qnolinkcmds, etc.
>>>
>>>
>>> https://gcc.gnu.org/git/?p=gcc.git;a=commitdiff;h=602fa1e9d3ea5e87d4d6e17e3e91fc2647e42da3
>>>
>>> - [DONE] This email thread backports the above patch to
>>> rtems-source-builder so that we can use it in GCC 7.3. Chris accepted
>>> this
>>> patch into rtems-source-builder here
>>>
>>>
>>> https://git.rtems.org/rtems-source-builder/commit/?id=defa958301215995b0fa41d8e65cb23c9a28a847
>>>
>>> - [PENDING - Joel] I recently made a GCC patch to have GCC build crti.o
>>> and
>>> crtn.o for the x86-64 RTEMS tools, so we can have symbols like "_fini"
>>> added automatically through our bsp_specs file, the way we do for other
>>> architectures. As Sebastian mentioned in an email to you and me, I
>>> believe
>>> he'd prefer if you reviewed the patch here:
>>> https://gcc.gnu.org/ml/gcc-patches/2018-05/msg00126.html
>>>
>>> - [PENDING - Amaan, and later Chris] Once the crti.o patch above is
>>> merged
>>> into GCC, I'll make a corresponding backport patch to
>>> rtems-source-builder
>>> to make that available to GCC 7.3 as well, and then Chris can review +
>>> merge that in after.
>>>
>> I meant to deal with this today.
>>
>> I am leaving myself a note to get this merged next week. Ping me in the
>> middle
>> of next week if you haven't heard action.
>
>
> This is now committed on the gcc master.
>
> https://gcc.gnu.org/ml/gcc-cvs/2018-05/msg00226.html
>
> Hopefully that helps clean up the RSB.
>
>>
>>
>>>
>>> Hope that clears things up!
>>
>>
>> It does.
>>>
>>>
>>> > On Thu, May 3, 2018, 8:38 PM Chris Johns <chrisj at rtems.org> wrote:
>>>
>>> >> On 07/04/2018 08:35, Amaan Cheval wrote:
>>> >> > Adds -qnolinkcmds, -qrtems, and defaults to pass to linker, for
>>> >> > gcc targeting x86_64.
>>>
>>> >> Pushed, thank you.
>>>
>>> >> Chris
>>> >> _______________________________________________
>>> >> devel mailing list
>>> >> devel at rtems.org
>>> >> http://lists.rtems.org/mailman/listinfo/devel
>>>
>>> > _______________________________________________
>>> > devel mailing list
>>> > devel at rtems.org
>>> > http://lists.rtems.org/mailman/listinfo/devel
>>
>>
>
More information about the devel
mailing list