AArch64: Make processing less fragile in config.gcc
authorTamar Christina <tamar.christina@arm.com>
Fri, 26 Jul 2019 13:13:48 +0000 (13:13 +0000)
committerTamar Christina <tnfchris@gcc.gnu.org>
Fri, 26 Jul 2019 13:13:48 +0000 (13:13 +0000)
commit3644cadf6a9d5a5cd8e83b0123316cf184fa4e3e
tree16d58da2e9d42bb99ef86c2625cdcf4161f5f0cb
parent8f5331b25250d488910f62120f6b268486e99fbf
AArch64: Make processing less fragile in config.gcc

Due to config.gcc all the options need to be on one line because of the grep
lines which would select only the first line of the option.

This causes it not to select the right bits on options that are spread over
multiple lines when the --with-arch configure option is used.  The issue happens
silently and you just get a compiler with an incorrect set of default flags.

The current rules are quite rigid:

   1) No space between the AARCH64_OPT_EXTENSION and the opening (.
   2) No space between the opening ( and the extension name.
   3) No space after the extension name before the ,.
   4) Spaces are only allowed after a , and around |.

This patch makes this a lot less fragile by using the C pre-processor to flatten
the list and then provides much more flexible regex using group matching to
process the options instead of string replacement.  This removes all the
restrictions above and makes the code a bit more readable.

gcc/ChangeLog:

PR target/89517
* config.gcc: Relax parsing of AARCH64_OPT_EXTENSION.
* config/aarch64/aarch64-option-extensions.def: Add new comments
and restore easier to read options.

From-SVN: r273827
gcc/ChangeLog
gcc/config.gcc
gcc/config/aarch64/aarch64-option-extensions.def