This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Testcase g++.old-deja/g++.other/enum5.C on 3.4 branch
- From: Gabriel Dos Reis <gdr at integrable-solutions dot net>
- To: "Kaveh R. Ghazi" <ghazi at caipclassic dot rutgers dot edu>
- Cc: bje at redhat dot com, pinskia at gcc dot gnu dot org, gcc-patches at gcc dot gnu dot org
- Date: 02 Dec 2005 10:53:11 +0100
- Subject: Re: Testcase g++.old-deja/g++.other/enum5.C on 3.4 branch
- References: <200512020145.jB21jJvr016131@caipclassic.rutgers.edu>
"Kaveh R. Ghazi" <ghazi@caipclassic.rutgers.edu> writes:
| Gaby,
|
| Testcase g++.old-deja/g++.other/enum5.C on the 3.4 branch fails with
| this:
|
| WARNING: g++.old-deja/g++.other/enum5.C compilation failed to produce executable
|
| Applying part of Ben's last patch to the testcase allows it to
| compile,
|
| > 2004-11-04 Ben Elliston <bje@au.ibm.com>
| >
| > * g++.old-deja/g++.other/enum5.C (enum conditions): Move the
| > packed attribute to the definition to satisfy the new C++ parser.
|
| but gcc-3.4.x won't pack the enums and the runtime tests checking the
| sizes abort.
|
| > enum5.C:9: warning: `packed' attribute ignored
| > enum5.C:10: warning: `packed' attribute ignored
| > enum5.C:11: warning: `packed' attribute ignored
|
| PR 19997 was filed against the 3.4 branch for this, but closed by
| Andrew because it's fixed on 4.0. What would you like to do about
| this in 3.4.6?
|
| Shall I delete the testcasein 3.4?
I would prefer the testsuite tries jst to compile it instead of
running it; so a solution would be to remove the first line
// { dg-do run }
-- Gaby