driver: fix validate_switches logic
authorJason Merrill <jason@redhat.com>
Sun, 27 Nov 2022 19:30:14 +0000 (14:30 -0500)
committerJason Merrill <jason@redhat.com>
Fri, 2 Dec 2022 15:38:32 +0000 (10:38 -0500)
commit6d3c634c8baebd9ff12c39d61947752486758bd3
tree9831db43bb8060835be83e840e7adc83c20fd0ae
parent70596a0fb2a2ec072e1e97e37616e05041dfa4e5
driver: fix validate_switches logic

Under the old logic for validate_switches, once suffix or starred got set,
they stayed set for all later switches found in the spec.  So for e.g.

%{g*:%{%:debug-level-gt(0):

Once we see g*, starred is set.  Then we see %:, and it sees that as a
zero-length switch, which because starred is still set, matches any and all
command-line options.  So targets that use such a spec accept all options in
the driver, while ones that don't reject some, such as the recent
-nostdlib++.

This patch fixes the inconsistency, so all targets would complain about
-nostdlib++, and then sets SKIPOPT for it so they don't.

gcc/ChangeLog:

* gcc.cc (validate_switches): Reset suffix/starred on loop.

gcc/cp/ChangeLog:

* g++spec.cc (lang_specific_driver): Set SKIPOPT for nostdlib++.
gcc/cp/g++spec.cc
gcc/gcc.cc