On 14/10/15 16:24, Andreas Schwab wrote:
Jiong Wang <jiong.w...@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.w...@arm.com>

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


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.-]\+//'`
 
 			if [ $which = arch ]; then
 			  def=aarch64-arches.def

Reply via email to