| Message ID | CAD57uCfu91nx-oPb4v9M+UUX7qn=cqfZYiODdV5jgQvC_cOwsA@mail.gmail.com |
|---|---|
| State | New |
| Headers | show |
Hi Yvan, On 13/01/16 14:41, Yvan Roux wrote: > Hi, > > There is a small typo into arm-arches.def which prevent the compiler > to be configured for armv8.1-a and armv8.1-a+crc, config.gcc and > genopt.sh assume no whitespace up to the first "," in each entry. > > I think that this change is obvious, but is it ok for trunk ? > > (Tested by building an armv8l-linux-gnueabihf compiler configured for armv8.1-a) > > Thanks > Yvan > > 2016-01-13 Yvan Roux <yvan.roux@linaro.org> > > * config/arm/arm-arches.def: Remove spurious whitespace in "armv8.1-a" > and "armv8.1-a+crc" entries. This is ok. Thanks for catching this, Kyrill
diff --git a/gcc/config/arm/arm-arches.def b/gcc/config/arm/arm-arches.def index 59633c4..fd02b18 100644 --- a/gcc/config/arm/arm-arches.def +++ b/gcc/config/arm/arm-arches.def @@ -57,9 +57,9 @@ ARM_ARCH("armv7-m", cortexm3, 7M, ARM_FSET_MAKE_CPU1 (FL_CO_PROC | FL_FOR_ ARM_ARCH("armv7e-m", cortexm4, 7EM, ARM_FSET_MAKE_CPU1 (FL_CO_PROC | FL_FOR_ARCH7EM)) ARM_ARCH("armv8-a", cortexa53, 8A, ARM_FSET_MAKE_CPU1 (FL_CO_PROC | FL_FOR_ARCH8A)) ARM_ARCH("armv8-a+crc",cortexa53, 8A, ARM_FSET_MAKE_CPU1 (FL_CO_PROC | FL_CRC32 | FL_FOR_ARCH8A)) -ARM_ARCH ("armv8.1-a", cortexa53, 8A, +ARM_ARCH("armv8.1-a", cortexa53, 8A, ARM_FSET_MAKE (FL_CO_PROC | FL_FOR_ARCH8A, FL2_FOR_ARCH8_1A)) -ARM_ARCH ("armv8.1-a+crc",cortexa53, 8A, +ARM_ARCH("armv8.1-a+crc",cortexa53, 8A, ARM_FSET_MAKE (FL_CO_PROC | FL_CRC32 | FL_FOR_ARCH8A, FL2_FOR_ARCH8_1A)) ARM_ARCH("iwmmxt", iwmmxt, 5TE, ARM_FSET_MAKE_CPU1 (FL_LDSCHED | FL_STRONG | FL_FOR_ARCH5TE | FL_XSCALE | FL_IWMMXT))
Hi, There is a small typo into arm-arches.def which prevent the compiler to be configured for armv8.1-a and armv8.1-a+crc, config.gcc and genopt.sh assume no whitespace up to the first "," in each entry. I think that this change is obvious, but is it ok for trunk ? (Tested by building an armv8l-linux-gnueabihf compiler configured for armv8.1-a) Thanks Yvan 2016-01-13 Yvan Roux <yvan.roux@linaro.org> * config/arm/arm-arches.def: Remove spurious whitespace in "armv8.1-a" and "armv8.1-a+crc" entries.