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 <j...@rtems.org> wrote: > > > On Fri, May 4, 2018 at 4:25 PM, Joel Sherrill <j...@rtems.org> wrote: >> >> >> >> On Fri, May 4, 2018 at 4:58 AM, Amaan Cheval <amaan.che...@gmail.com> >> wrote: >>> >>> On Fri, May 4, 2018 at 7:54 AM Joel Sherrill <j...@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 <chr...@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@rtems.org >>> >> http://lists.rtems.org/mailman/listinfo/devel >>> >>> > _______________________________________________ >>> > devel mailing list >>> > devel@rtems.org >>> > http://lists.rtems.org/mailman/listinfo/devel >> >> > _______________________________________________ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/devel
