diff options
| author | Nathan Sidwell <nathan@codesourcery.com> | 2003-09-18 17:07:53 +0000 |
|---|---|---|
| committer | Nathan Sidwell <nathan@gcc.gnu.org> | 2003-09-18 17:07:53 +0000 |
| commit | 85b22f7857a745d8ec12d88d61b770a598208ecf (patch) | |
| tree | e282a8b31bbf685e2f19352bd0b60a68eddc7257 /gcc/cp/optimize.c | |
| parent | f857cb427e56de666cafa31769bb6208358c374c (diff) | |
| download | gcc-85b22f7857a745d8ec12d88d61b770a598208ecf.zip gcc-85b22f7857a745d8ec12d88d61b770a598208ecf.tar.gz gcc-85b22f7857a745d8ec12d88d61b770a598208ecf.tar.bz2 | |
re PR c++/9848 (missing 'unused parameter' warning.)
cp:
PR c++/9848
* optimize.c (maybe_clone_body): Don't set MARK_USED on parameters
here.
* semantics.c (expand_body): Set it here on the remaining clones.
testsuite:
PR c++/9848
* g++.dg/warn/Wunused-4.C: New test.
From-SVN: r71528
Diffstat (limited to 'gcc/cp/optimize.c')
| -rw-r--r-- | gcc/cp/optimize.c | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/gcc/cp/optimize.c b/gcc/cp/optimize.c index 3204311..2b7df6c 100644 --- a/gcc/cp/optimize.c +++ b/gcc/cp/optimize.c @@ -120,7 +120,6 @@ bool maybe_clone_body (tree fn) { tree clone; - bool first = true; /* We only clone constructors and destructors. */ if (!DECL_MAYBE_IN_CHARGE_CONSTRUCTOR_P (fn) @@ -139,7 +138,7 @@ maybe_clone_body (tree fn) list. */ for (clone = TREE_CHAIN (fn); clone && DECL_CLONED_FUNCTION_P (clone); - clone = TREE_CHAIN (clone), first = false) + clone = TREE_CHAIN (clone)) { tree parm; tree clone_parm; @@ -175,13 +174,8 @@ maybe_clone_body (tree fn) clone_parm = TREE_CHAIN (clone_parm); for (; parm; parm = TREE_CHAIN (parm), clone_parm = TREE_CHAIN (clone_parm)) - { - /* Update this parameter. */ - update_cloned_parm (parm, clone_parm); - /* We should only give unused information for one clone. */ - if (!first) - TREE_USED (clone_parm) = 1; - } + /* Update this parameter. */ + update_cloned_parm (parm, clone_parm); /* Start processing the function. */ push_to_top_level (); |