Commit 49f3f450 by Wilco Dijkstra Committed by Wilco Dijkstra

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
	* opts.c (parse_and_check_align_values): Allow 4 alignment values.
M    gcc/ChangeLog
M    gcc/opts.c

From-SVN: r271864
parent 72215009
2019-06-03 Wilco Dijkstra <wdijkstr@arm.com>
PR driver/90684
* opts.c (parse_and_check_align_values): Allow 4 alignment values.
2019-06-03 Kyrylo Tkachov <kyrylo.tkachov@arm.com> 2019-06-03 Kyrylo Tkachov <kyrylo.tkachov@arm.com>
* config/aarch64/iterators.md (MAX_OPP): New code attr. * config/aarch64/iterators.md (MAX_OPP): New code attr.
......
...@@ -2022,14 +2022,7 @@ parse_and_check_align_values (const char *flag, ...@@ -2022,14 +2022,7 @@ parse_and_check_align_values (const char *flag,
free (str); free (str);
/* Check that we have a correct number of values. */ /* Check that we have a correct number of values. */
#ifdef SUBALIGN_LOG if (result_values.is_empty () || result_values.length () > 4)
unsigned max_valid_values = 4;
#else
unsigned max_valid_values = 2;
#endif
if (result_values.is_empty ()
|| result_values.length () > max_valid_values)
{ {
if (report_error) if (report_error)
error_at (loc, "invalid number of arguments for %<-falign-%s%> " error_at (loc, "invalid number of arguments for %<-falign-%s%> "
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment