--- Comment #22 from manu at gcc dot gnu dot org 2007-06-30 12:58 ---
Fixed for GCC 4.3
--
manu at gcc dot gnu dot org changed:
What|Removed |Added
Status|NEW
--- Comment #21 from manu at gcc dot gnu dot org 2007-06-30 12:56 ---
Subject: Bug 4076
Author: manu
Date: Sat Jun 30 12:56:43 2007
New Revision: 126144
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=126144
Log:
2007-06-30 Manuel Lopez-Ibanez <[EMAIL PROTECTED]>
PR c/
--- Comment #20 from mrs at apple dot com 2007-06-19 23:36 ---
The patch was approved today on the gcc-patches list.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=4076
--- Comment #19 from patchapp at dberlin dot org 2007-05-20 02:10 ---
Subject: Bug number PR4076
A patch for this bug has been added to the patch tracker.
The mailing list url for the patch is
http://gcc.gnu.org/ml/gcc-patches/2007-05/msg01298.html
--
http://gcc.gnu.org/bugzilla/sh
--- Comment #18 from manu at gcc dot gnu dot org 2007-04-13 22:54 ---
(In reply to comment #17)
> Manuel has a good patch for this. I don't know what's holding it up.
>
Some issues were raised, in particular, my patch doesn't warn about functions
in anonymous namespaces:
http://gcc.gn
--- Comment #17 from steven at gcc dot gnu dot org 2007-04-13 22:39 ---
Manuel has a good patch for this. I don't know what's holding it up.
--
steven at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #16 from patchapp at dberlin dot org 2007-03-03 11:50 ---
Subject: Bug number PR c/4076
A patch for this bug has been added to the patch tracker.
The mailing list url for the patch is
http://gcc.gnu.org/ml/gcc-patches/2007-03/msg00171.html
--
http://gcc.gnu.org/bugzilla
--- Comment #15 from manu at gcc dot gnu dot org 2007-02-16 23:35 ---
(In reply to comment #14)
> (In reply to comment #12)
> > 1) there is no warning if I add the keyword "inline" to objc-list.h
> > (list_free).
>
> I preapprove the patch which adds the keyword inline to that function
--- Comment #14 from pinskia at gcc dot gnu dot org 2007-02-16 21:33
---
(In reply to comment #12)
> 1) there is no warning if I add the keyword "inline" to objc-list.h
> (list_free).
I preapprove the patch which adds the keyword inline to that function.
Thanks,
Andrew Pinski
--
--- Comment #13 from mrs at apple dot com 2007-02-16 17:59 ---
Adding inline seems obvious to me, all the other functions are marked inline.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=4076
--- Comment #12 from manu at gcc dot gnu dot org 2007-02-16 09:53 ---
I get warning for gcc/testsuite/objc.dg/headers.m
/home/manuel/src/trunk/gcc/testsuite/../../libobjc/objc/objc-list.h:144:
warning: 'list_free' defined but not used
What should I do about this?
1) there is no warnin
--- Comment #11 from stevenb dot gcc at gmail dot com 2007-01-29 18:22
---
Subject: Re: -Wunused doesn't warn about static function only called by
itself.
If it is unused, don't hesitate to remove it. :-)
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=4076
--- Comment #10 from manu at gcc dot gnu dot org 2007-01-29 18:08 ---
(In reply to comment #9)
> I will submit two patches. The first one will remove the function. Then, I
> have
> to regtest the second one: perhaps we find another unused function!
Another one:
genautomata.c: static i
--- Comment #9 from manu at gcc dot gnu dot org 2007-01-28 17:23 ---
I will submit two patches. The first one will remove the function. Then, I have
to regtest the second one: perhaps we find another unused function!
Steven, if you really think that the case:
static void foo(void)
{
--- Comment #8 from stevenb dot gcc at gmail dot com 2007-01-27 19:58
---
Subject: Re: -Wunused doesn't warn about static function only called by
itself.
Just one for everything should suffice.
Or, if you prefer, you can remove that one function with a separate
patch first, which, I
--- Comment #7 from manu at gcc dot gnu dot org 2007-01-27 16:04 ---
So then, should I prepare two separated patches or just one for everything ?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=4076
--- Comment #6 from hubicka at gcc dot gnu dot org 2007-01-27 15:57 ---
update_inlined_to_pointers is obviously no longer needed and can be safely
removed now. Thanks for noticing it ;)
Honza
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=4076
--- Comment #5 from manu at gcc dot gnu dot org 2007-01-27 15:46 ---
Anyway, here is my current patch, perhaps someone can find some use for it:
Index: gcc/testsuite/gcc.dg/Wunused-function.c
===
--- gcc/testsuite/gcc.dg/Wu
--- Comment #4 from manu at gcc dot gnu dot org 2007-01-27 15:45 ---
The funny thing is that if we fix this, bootstrap will break (with a warning
treated as an error) on tree-optimize.c (update_inlined_to_pointers) since it
seems that that function is only used by itself.
Is it dead cod
--
steven at gcc dot gnu dot org changed:
What|Removed |Added
CC|stevenb at suse dot de |
AssignedTo|unassigned at gcc dot gnu |steven at gcc dot gnu
--
What|Removed |Added
CC||pinskia at gcc dot gnu dot
||org
Last reconfirmed|2004-08-04 0
21 matches
Mail list logo
