This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PR33713: remove -fforce-addr
On Dec 14, 2007 2:57 AM, Hans-Peter Nilsson <hans-peter.nilsson@axis.com> wrote:
> > Date: Tue, 4 Dec 2007 19:13:35 -0400
> > From: Aldy Hernandez <aldyh@redhat.com>
>
> > > Anyway, the point of this communication: was it really a good
> > > thing to remove all test-cases using -fforce-addr? Why not just
> > > remove their -fforce-addr argument? Can we add them back?
> >
> > I don't have a problem with that. Steven?
>
> Any news on this?
IMHO without -fforce-addr the test cases test nothing that isn't
already triggered by a gcc build or by another test case. I see no
point in keeping aruond test cases that add no test coverage.
Gr.
Steven
Index: testsuite/gcc.c-torture/compile/20050802-1.c
===================================================================
--- testsuite/gcc.c-torture/compile/20050802-1.c (revision 130511)
+++ testsuite/gcc.c-torture/compile/20050802-1.c (working copy)
@@ -1,10 +0,0 @@
-/* PR 23196 */
-/* { dg-options "-fforce-addr" } */
-
-void foo()
-{
- char c;
-
- c |= 1;
- bar(&c);
-}
Index: testsuite/gcc.dg/20011113-1.c
===================================================================
--- testsuite/gcc.dg/20011113-1.c (revision 130511)
+++ testsuite/gcc.dg/20011113-1.c (working copy)
@@ -1,16 +0,0 @@
-/* Copyright (C) 2001 Free Software Foundation.
- by Hans-Peter Nilsson <hp@axis.com> */
-
-/* { dg-do compile } */
-/* { dg-options "-O2 -fforce-addr" } */
-
-const char foo[] = "fum";
-const struct fi
-{
- const char *const in;
- const char *const out;
- const int flags;
-} fie[] = {
- {"nw", " new", 0},
- {"dl", foo, 1}
-};