| Message ID | 1410814919-11523-5-git-send-email-thomas.petazzoni@free-electrons.com |
|---|---|
| State | Accepted |
| Commit | fd828fd98fecb5b3baf529c8dc6fbd2971e2f172 |
| Headers | show |
Thomas, All, On 2014-09-15 23:01 +0200, Thomas Petazzoni spake thusly: > On ARM, we were defining both the CPU type and the architecture > variant. However, depending on the version of gcc, a given combination > of (CPU, architecture) may not be the same. Since the architecture > variant is implied by the CPU type, given the former is not necessary, > and we can simply specify the latter. > > From the gcc documentation: > > This specifies the name of the target ARM processor. GCC uses this > name to derive the name of the target ARM architecture (as if > specified by -march) and the ARM processor type for which to tune > for performance (as if specified by -mtune). Where this option is > used in conjunction with -march or -mtune, those options take > precedence over the appropriate part of this option. > > Note that we verified that for all BR2_GCC_TARGET_ARCH value that > existed, a proper BR2_GCC_TARGET_CPU value is defined. I guess we'll keep this one in limbo until we have a final answer on switching from -mcpu to -mtune for ARM. Regards, Yann E. MORIN. > Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> > --- > arch/Config.in.arm | 19 ------------------- > 1 file changed, 19 deletions(-) > > diff --git a/arch/Config.in.arm b/arch/Config.in.arm > index 6e3c4e6..68f6370 100644 > --- a/arch/Config.in.arm > +++ b/arch/Config.in.arm > @@ -352,25 +352,6 @@ config BR2_GCC_TARGET_CPU > default "xscale" if BR2_xscale > default "iwmmxt" if BR2_iwmmxt > > -config BR2_GCC_TARGET_ARCH > - default "armv4t" if BR2_arm920t > - default "armv4t" if BR2_arm922t > - default "armv5te" if BR2_arm926t > - default "armv6j" if BR2_arm1136jf_s > - default "armv6zk" if BR2_arm1176jz_s > - default "armv6zk" if BR2_arm1176jzf_s > - default "armv7-a" if BR2_cortex_a5 > - default "armv7-a" if BR2_cortex_a7 > - default "armv7-a" if BR2_cortex_a8 > - default "armv7-a" if BR2_cortex_a9 > - default "armv7-a" if BR2_cortex_a12 > - default "armv7-a" if BR2_cortex_a15 > - default "armv4" if BR2_fa526 > - default "armv7-a" if BR2_pj4 > - default "armv4" if BR2_strongarm > - default "armv5te" if BR2_xscale > - default "iwmmxt" if BR2_iwmmxt > - > config BR2_GCC_TARGET_ABI > default "aapcs-linux" > > -- > 2.0.0 >
diff --git a/arch/Config.in.arm b/arch/Config.in.arm index 6e3c4e6..68f6370 100644 --- a/arch/Config.in.arm +++ b/arch/Config.in.arm @@ -352,25 +352,6 @@ config BR2_GCC_TARGET_CPU default "xscale" if BR2_xscale default "iwmmxt" if BR2_iwmmxt -config BR2_GCC_TARGET_ARCH - default "armv4t" if BR2_arm920t - default "armv4t" if BR2_arm922t - default "armv5te" if BR2_arm926t - default "armv6j" if BR2_arm1136jf_s - default "armv6zk" if BR2_arm1176jz_s - default "armv6zk" if BR2_arm1176jzf_s - default "armv7-a" if BR2_cortex_a5 - default "armv7-a" if BR2_cortex_a7 - default "armv7-a" if BR2_cortex_a8 - default "armv7-a" if BR2_cortex_a9 - default "armv7-a" if BR2_cortex_a12 - default "armv7-a" if BR2_cortex_a15 - default "armv4" if BR2_fa526 - default "armv7-a" if BR2_pj4 - default "armv4" if BR2_strongarm - default "armv5te" if BR2_xscale - default "iwmmxt" if BR2_iwmmxt - config BR2_GCC_TARGET_ABI default "aapcs-linux"
On ARM, we were defining both the CPU type and the architecture variant. However, depending on the version of gcc, a given combination of (CPU, architecture) may not be the same. Since the architecture variant is implied by the CPU type, given the former is not necessary, and we can simply specify the latter. From the gcc documentation: This specifies the name of the target ARM processor. GCC uses this name to derive the name of the target ARM architecture (as if specified by -march) and the ARM processor type for which to tune for performance (as if specified by -mtune). Where this option is used in conjunction with -march or -mtune, those options take precedence over the appropriate part of this option. Note that we verified that for all BR2_GCC_TARGET_ARCH value that existed, a proper BR2_GCC_TARGET_CPU value is defined. Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> --- arch/Config.in.arm | 19 ------------------- 1 file changed, 19 deletions(-)