This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Add support for ARM embedded multilibs
- From: "Jasmin J." <jasmin at anw dot at>
- To: Ramana Radhakrishnan <ramana dot gcc at googlemail dot com>
- Cc: gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 4 Nov 2015 09:43:59 +0100
- Subject: Re: [PATCH] Add support for ARM embedded multilibs
- Authentication-results: sourceware.org; auth=none
- References: <56395178 dot 8000006 at anw dot at> <CAJA7tRbQPN2HDtP3HjASn6W5s=Lg83k9+DDDJ92_GjDAs95BBw at mail dot gmail dot com>
Hello Ramana!
> Thank you for your patch - In this case before you make any more
> changes to this patch - comparing your patch and Terry's patch here
> https://gcc.gnu.org/ml/gcc-patches/2014-05/msg00729.html shows no real
> differences
As I wrote in the patch, it is a port from the embedded-4_9-branch and it
is exactly what Terry did. I didn't know this patch. I extracted it from
the branch.
All other stuff required to build a gcc for the ARM embedded CPUs are already
on gcc trunk. This last piece is missing to get rid of the embedded-X_X-branch!
I would love to see it in mainline GCC.
> I would like to ask if you have a copyright assignment on file with the FSF
I will do that.
> How was your patch tested
The patch is old an doesn't contain any real GCC change, but support for the
library building process only. This was used since 15 months by many people
and I used it to build an ARM embedded compiler based on gcc-5-branch.
> see for example how I added t-aprofile to the backend and the kind of
> testing it underwent
I will look on that, if it is really required.
>> * configure.ac (with_multilib_list): Export for being used in arm
>> embedded multilib fragment.
>> * Makefile.in (with_multilib_list): Import for being used in
>> multilib fragment.
>
> This is already being used in config.gcc - why do you need this
> additional hunk ?
To be hones, I ported the patch and checked if it works. I will analyse it more
detailed, if this is really required.
> The t-rmprofile file will need updating to newer values for -mcpu and
> march as well as comments up top to explain what multilibs are being
> built .
Where can I find them?
BR
Jasmin