[AArch64] --with-arch in config.gcc support "."

Jiong Wang jiong.wang@arm.com
Thu Oct 15 08:58:00 GMT 2015


On 14/10/15 16:24, Andreas Schwab wrote:
> Jiong Wang <jiong.wang@arm.com> writes:
>
>> diff --git a/gcc/config.gcc b/gcc/config.gcc
>> index 5818663..215ad9a 100644
>> --- a/gcc/config.gcc
>> +++ b/gcc/config.gcc
>> @@ -3544,7 +3544,7 @@ case "${target}" in
>>   
>>   			eval "val=\$with_$which"
>>   			base_val=`echo $val | sed -e 's/\+.*//'`
>> -			ext_val=`echo $val | sed -e 's/[a-z0-9\-]\+//'`
>> +			ext_val=`echo $val | sed -e 's/[a-z0-9\.\-]\+//'`
> Neither backslash is needed inside bracket expressions (in fact, the
> backslash is taken literally here) as the set of special characters is
> completely different there.

Thanks for pointing this out.
I am surprised "-" doesn't need escape as it serve as range operator. Maybe
sed will make it an operator when it's following number or alphabetic.

Anyway, I tested below patch, which still parse the extension correctly.

OK for trunk?
  
2015-10-15  Jiong Wang  <jiong.wang@arm.com>

gcc/
   * config.gcc: Recognize "." in architecture base name for AArch64.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: k.patch
Type: text/x-diff
Size: 401 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20151015/720fda19/attachment.bin>


More information about the Gcc-patches mailing list