From: Wilco Dijkstra <wdijkstr@arm.com>
Date: Tue, 16 Jul 2019 16:17:39 +0000 (+0000)
Subject: Fix alignment option parser (PR90684)
X-Git-Tag: releases/gcc-9.2.0~141
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=93750effc52353765394bff949a34f81f16a4673;p=thirdparty%2Fgcc.git

Fix alignment option parser (PR90684)

Fix the alignment option parser to always allow up to 4 alignments.
Now -falign-functions=16:8:8:8 no longer reports an error.

    gcc/
	PR driver/90684
	* gcc/opts.c (parse_and_check_align_values): Allow 4 alignment values.

From-SVN: r273533
---

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 063ae505e7b0..c2d1caa81b99 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,11 @@
+2019-07-16  Wilco Dijkstra  <wdijkstr@arm.com>
+
+	Backport from mainline
+	2019-05-30  Wilco Dijkstra  <wdijkstr@arm.com>
+
+	PR driver/90684
+	* gcc/opts.c (parse_and_check_align_values): Allow 4 alignment values.
+
 2019-07-14  Jakub Jelinek  <jakub@redhat.com>
 
 	Backported from mainline
diff --git a/gcc/opts.c b/gcc/opts.c
index 6d6ff19de2d9..6164d49fa08f 100644
--- a/gcc/opts.c
+++ b/gcc/opts.c
@@ -2016,14 +2016,7 @@ parse_and_check_align_values (const char *flag,
   free (str);
 
   /* Check that we have a correct number of values.  */
-#ifdef SUBALIGN_LOG
-  unsigned max_valid_values = 4;
-#else
-  unsigned max_valid_values = 2;
-#endif
-
-  if (result_values.is_empty ()
-      || result_values.length () > max_valid_values)
+  if (result_values.is_empty () || result_values.length () > 4)
     {
       if (report_error)
 	error_at (loc, "invalid number of arguments for %<-falign-%s%> "