Re: PR33713: remove -fforce-addr

["Steven Bosscher" <stevenb.gcc@gmail.com>]

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}
-};