diff options
author | Matt Kraai <kraai@alumni.cmu.edu> | 2004-11-29 00:37:54 +0000 |
---|---|---|
committer | Matt Kraai <kraai@gcc.gnu.org> | 2004-11-29 00:37:54 +0000 |
commit | a2163d83a300310199ae507af05691166a8287dd (patch) | |
tree | c7bfd8aba2f8ca93263b770a18023dfb54d9a4c1 /gcc | |
parent | 77109b506986bd02bd9ba811e270407bf3b9470c (diff) | |
download | gcc-a2163d83a300310199ae507af05691166a8287dd.zip gcc-a2163d83a300310199ae507af05691166a8287dd.tar.gz gcc-a2163d83a300310199ae507af05691166a8287dd.tar.bz2 |
re PR other/17594 (GCC does not error about unknown options which starts with a valid option)
(gcc)
PR diagnostic/17594
* opts.c (find_opt): Require that the input match a switch exactly
or that the switch take a joined option to be interpreted as a
match for another language.
(gcc/testsuite)
* gcc.dg/funroll-loops-all.c: New.
From-SVN: r91444
Diffstat (limited to 'gcc')
-rw-r--r-- | gcc/ChangeLog | 7 | ||||
-rw-r--r-- | gcc/opts.c | 12 | ||||
-rw-r--r-- | gcc/testsuite/ChangeLog | 4 | ||||
-rw-r--r-- | gcc/testsuite/gcc.dg/funroll-loops-all.c | 4 |
4 files changed, 21 insertions, 6 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 2528870..73e6ad9 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,10 @@ +2004-11-28 Matt Kraai <kraai@alumni.cmu.edu> + + PR diagnostic/17594 + * opts.c (find_opt): Require that the input match a switch exactly + or that the switch take a joined option to be interpreted as a + match for another language. + 2004-11-28 Andrew Pinski <pinskia@physics.uc.edu> * timevar.def (TV_TREE_REDPHI): New timevar. @@ -176,13 +176,13 @@ find_opt (const char *input, int lang_mask) { const struct cl_option *opt = &cl_options[mn]; - /* Is this switch a prefix of the input? */ - if (!strncmp (input, opt->opt_text + 1, opt->opt_len)) + /* Is the input either an exact match or a prefix that takes a + joined argument? */ + if (!strncmp (input, opt->opt_text + 1, opt->opt_len) + && (input[opt->opt_len] == '\0' || (opt->flags & CL_JOINED))) { - /* If language is OK, and the match is exact or the switch - takes a joined argument, return it. */ - if ((opt->flags & lang_mask) - && (input[opt->opt_len] == '\0' || (opt->flags & CL_JOINED))) + /* If language is OK, return it. */ + if (opt->flags & lang_mask) return mn; /* If we haven't remembered a prior match, remember this diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 8e56d03..94174d9 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,7 @@ +2004-11-28 Matt Kraai <kraai@alumni.cmu.edu> + + * gcc.dg/funroll-loops-all.c: New. + 2004-11-28 Hans-Peter Nilsson <hp@bitrange.com> PR target/18489 diff --git a/gcc/testsuite/gcc.dg/funroll-loops-all.c b/gcc/testsuite/gcc.dg/funroll-loops-all.c new file mode 100644 index 0000000..c627675 --- /dev/null +++ b/gcc/testsuite/gcc.dg/funroll-loops-all.c @@ -0,0 +1,4 @@ +/* PR 17594 */ +/* { dg-do compile } */ +/* { dg-options "-funroll-loops-all" } */ +/* { dg-error "unrecognized command line option \"-funroll-loops-all\"" "" { target *-*-* } 0 } */ |