This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix PR78333
- From: Rainer Orth <ro at CeBiTec dot Uni-Bielefeld dot DE>
- To: Christophe Lyon <christophe dot lyon at linaro dot org>
- Cc: Richard Biener <rguenther at suse dot de>, "gcc-patches\@gcc.gnu.org" <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 17 Nov 2016 09:56:47 +0100
- Subject: Re: [PATCH] Fix PR78333
- Authentication-results: sourceware.org; auth=none
- References: <alpine.LSU.2.11.1611161032510.5294@t29.fhfr.qr> <CAKdteOaCNTY4P=ga0XqWiDPLYG7h=oYaejecqMWrWOPXy865AA@mail.gmail.com>
Hi Christophe,
> The new testcase fails to link on bare-metal targets (arm/aarch64):
> /cc4ckSGA.o: In function `main':
> pr78333.c:(.text+0x1c): undefined reference to `__cyg_profile_func_enter'
> pr78333.c:(.text+0x34): undefined reference to `__cyg_profile_func_exit'
> collect2: error: ld returned 1 exit status
>
> I understand that we really want this test to have dg-do link unlike most
> of the other tests using -finstrument-functions, so we do not want to
> switch to dg-do compile.
>
> Do we want to skip this test in bare-metal configs, or is it a new bug?
it's not only bare-metal targets, but probably most non-glibc targets.
E.g the test also FAILs on Solaris in the same way, unlike Linux/x86_64
where glibc provides those functions.
Probably providing dummy implemenations as in the original testcase in
the PR is enough?
Rainer
--
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University