* [PATCH, GCC/ARM] Add multilib mapping for Cortex-M23 & Cortex-M33
@ 2016-11-30 11:48 Thomas Preudhomme
2016-11-30 11:52 ` Ramana Radhakrishnan
2016-12-01 10:10 ` [arm-embedded] " Thomas Preudhomme
0 siblings, 2 replies; 3+ messages in thread
From: Thomas Preudhomme @ 2016-11-30 11:48 UTC (permalink / raw)
To: Kyrill Tkachov, Ramana Radhakrishnan, Richard Earnshaw, gcc-patches
[-- Attachment #1: Type: text/plain, Size: 756 bytes --]
Hi,
With ARM Cortex-M23 and Cortex-M33 and the support for RM profile multilib added
recently, it's time to add the corresponding CPU to architecture mappings in
config/arm/t-rmprofile. Note that Cortex-M33 is mapped to ARMv8-M Mainline
because there is no transitive closure of mappings and the multilib for ARMv8-M
Mainline with DSP extensions is ARMv8-M Mainline.
ChangeLog entry is as follows:
*** gcc/ChangeLog ***
2016-11-30 Thomas Preud'homme <thomas.preudhomme@arm.com>
* config/arm/t-rmprofile: Add mappings for Cortex-M23 and Cortex-M33.
Testing: Linking fails before this patch when targeting one of these two cores
and using rmprofile multilib but succeeds with the patch.
Is this ok for stage3?
Best regards,
Thomas
[-- Attachment #2: add_m23_m33_multilib_mappings.patch --]
[-- Type: text/x-patch, Size: 779 bytes --]
diff --git a/gcc/config/arm/t-rmprofile b/gcc/config/arm/t-rmprofile
index c8b5c9cbd03694eea69855e20372afa3e97d6b4c..93aa909b4d942ad9875a95e0d4397ff17b317905 100644
--- a/gcc/config/arm/t-rmprofile
+++ b/gcc/config/arm/t-rmprofile
@@ -102,6 +102,8 @@ MULTILIB_MATCHES += march?armv6s-m=mcpu?cortex-m1.small-multiply
MULTILIB_MATCHES += march?armv7-m=mcpu?cortex-m3
MULTILIB_MATCHES += march?armv7e-m=mcpu?cortex-m4
MULTILIB_MATCHES += march?armv7e-m=mcpu?cortex-m7
+MULTILIB_MATCHES += march?armv8-m.base=mcpu?cortex-m23
+MULTILIB_MATCHES += march?armv8-m.main=mcpu?cortex-m33
MULTILIB_MATCHES += march?armv7=mcpu?cortex-r4
MULTILIB_MATCHES += march?armv7=mcpu?cortex-r4f
MULTILIB_MATCHES += march?armv7=mcpu?cortex-r5
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH, GCC/ARM] Add multilib mapping for Cortex-M23 & Cortex-M33
2016-11-30 11:48 [PATCH, GCC/ARM] Add multilib mapping for Cortex-M23 & Cortex-M33 Thomas Preudhomme
@ 2016-11-30 11:52 ` Ramana Radhakrishnan
2016-12-01 10:10 ` [arm-embedded] " Thomas Preudhomme
1 sibling, 0 replies; 3+ messages in thread
From: Ramana Radhakrishnan @ 2016-11-30 11:52 UTC (permalink / raw)
To: Thomas Preudhomme
Cc: Kyrill Tkachov, Ramana Radhakrishnan, Richard Earnshaw, gcc-patches
On Wed, Nov 30, 2016 at 11:48 AM, Thomas Preudhomme
<thomas.preudhomme@foss.arm.com> wrote:
> Hi,
>
> With ARM Cortex-M23 and Cortex-M33 and the support for RM profile multilib
> added recently, it's time to add the corresponding CPU to architecture
> mappings in config/arm/t-rmprofile. Note that Cortex-M33 is mapped to
> ARMv8-M Mainline because there is no transitive closure of mappings and the
> multilib for ARMv8-M Mainline with DSP extensions is ARMv8-M Mainline.
>
> ChangeLog entry is as follows:
>
>
> *** gcc/ChangeLog ***
>
> 2016-11-30 Thomas Preud'homme <thomas.preudhomme@arm.com>
>
> * config/arm/t-rmprofile: Add mappings for Cortex-M23 and
> Cortex-M33.
>
>
> Testing: Linking fails before this patch when targeting one of these two
> cores and using rmprofile multilib but succeeds with the patch.
>
> Is this ok for stage3?
This is OK despite stage3 as it only adds support for a
micro-architecture in a multilib fragment for library selection to
work properly when the CPU is on the command line.
Thanks,
Ramana
>
> Best regards,
>
> Thomas
^ permalink raw reply [flat|nested] 3+ messages in thread
* [arm-embedded] [PATCH, GCC/ARM] Add multilib mapping for Cortex-M23 & Cortex-M33
2016-11-30 11:48 [PATCH, GCC/ARM] Add multilib mapping for Cortex-M23 & Cortex-M33 Thomas Preudhomme
2016-11-30 11:52 ` Ramana Radhakrishnan
@ 2016-12-01 10:10 ` Thomas Preudhomme
1 sibling, 0 replies; 3+ messages in thread
From: Thomas Preudhomme @ 2016-12-01 10:10 UTC (permalink / raw)
To: gcc-patches
[-- Attachment #1: Type: text/plain, Size: 318 bytes --]
Hi,
We have decided to backport this patch to add multilib support for ARM
Cortex-M23 and Cortex-M33 to our embedded-6-branch.
*** gcc/ChangeLog ***
2016-11-30 Thomas Preud'homme <thomas.preudhomme@arm.com>
* config/arm/t-rmprofile: Add mappings for Cortex-M23 and Cortex-M33.
Best regards,
Thomas
[-- Attachment #2: [PATCH, GCC/ARM] Add multilib mapping for Cortex-M23 & Cortex-M33.eml --]
[-- Type: message/rfc822, Size: 5418 bytes --]
[-- Attachment #2.1.1: Type: text/plain, Size: 756 bytes --]
Hi,
With ARM Cortex-M23 and Cortex-M33 and the support for RM profile multilib added
recently, it's time to add the corresponding CPU to architecture mappings in
config/arm/t-rmprofile. Note that Cortex-M33 is mapped to ARMv8-M Mainline
because there is no transitive closure of mappings and the multilib for ARMv8-M
Mainline with DSP extensions is ARMv8-M Mainline.
ChangeLog entry is as follows:
*** gcc/ChangeLog ***
2016-11-30 Thomas Preud'homme <thomas.preudhomme@arm.com>
* config/arm/t-rmprofile: Add mappings for Cortex-M23 and Cortex-M33.
Testing: Linking fails before this patch when targeting one of these two cores
and using rmprofile multilib but succeeds with the patch.
Is this ok for stage3?
Best regards,
Thomas
[-- Attachment #2.1.2: add_m23_m33_multilib_mappings.patch --]
[-- Type: text/x-patch, Size: 779 bytes --]
diff --git a/gcc/config/arm/t-rmprofile b/gcc/config/arm/t-rmprofile
index c8b5c9cbd03694eea69855e20372afa3e97d6b4c..93aa909b4d942ad9875a95e0d4397ff17b317905 100644
--- a/gcc/config/arm/t-rmprofile
+++ b/gcc/config/arm/t-rmprofile
@@ -102,6 +102,8 @@ MULTILIB_MATCHES += march?armv6s-m=mcpu?cortex-m1.small-multiply
MULTILIB_MATCHES += march?armv7-m=mcpu?cortex-m3
MULTILIB_MATCHES += march?armv7e-m=mcpu?cortex-m4
MULTILIB_MATCHES += march?armv7e-m=mcpu?cortex-m7
+MULTILIB_MATCHES += march?armv8-m.base=mcpu?cortex-m23
+MULTILIB_MATCHES += march?armv8-m.main=mcpu?cortex-m33
MULTILIB_MATCHES += march?armv7=mcpu?cortex-r4
MULTILIB_MATCHES += march?armv7=mcpu?cortex-r4f
MULTILIB_MATCHES += march?armv7=mcpu?cortex-r5
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-12-01 10:10 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-11-30 11:48 [PATCH, GCC/ARM] Add multilib mapping for Cortex-M23 & Cortex-M33 Thomas Preudhomme
2016-11-30 11:52 ` Ramana Radhakrishnan
2016-12-01 10:10 ` [arm-embedded] " Thomas Preudhomme
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).