]> git.ipfire.org Git - thirdparty/gcc.git/commit
genmultilib: Fix sanity check
authorChristophe Lyon <christophe.lyon@arm.com>
Mon, 21 Nov 2022 11:12:14 +0000 (12:12 +0100)
committerChristophe Lyon <christophe.lyon@arm.com>
Mon, 21 Nov 2022 12:19:53 +0000 (13:19 +0100)
commitccb9c7b129206209cfc315ab1a0432b5f517bdd9
tree630953cffd27f6ad70cec75914ba0a895557bbbb
parent4c7d336b673df2f3bf23bc5e7a69c445a2320c04
genmultilib: Fix sanity check

My previous patch to add a sanity check to genmultilib actually
checked the number of dirnames with the number of "sets of options"
rather than the number of options, thus breaking the build on some
targets.

To avoid duplicating once more the loop that constructs the sed
patterns, this patch checks that the current dirname/osdirname is not
empty in the existing loops.

Are there targets where:
if [ "$1" != "${opt}" ]; then
is "legally" executed with an empty $1? (and thus where this patch
would incorrectly trigger an error?)

Sorry for the breakage. Tested on aarch64 by adding an option to
t-aarch64 and no corresponding dirname, and on x86_64.

gcc/ChangeLog:

* genmultilib: Fix options and dirnames/osdirnames sanity check.
gcc/genmultilib