public inbox for gcc@gcc.gnu.org
 help / color / mirror / Atom feed
* Switching the default for -fabi-version
@ 2003-10-22  0:49 Mark Mitchell
  2003-10-22  0:53 ` Joe Buck
                   ` (2 more replies)
  0 siblings, 3 replies; 33+ messages in thread
From: Mark Mitchell @ 2003-10-22  0:49 UTC (permalink / raw)
  To: gcc; +Cc: david.moore

Since nobody really objected to David Moore's suggestion that we make
-fabi-version=2 the default, I plan to change the default value of the
flag on the mainline in the next few days.

If you object, speak up now!

-- 
Mark Mitchell <mark@codesourcery.com>
CodeSourcery, LLC

^ permalink raw reply	[flat|nested] 33+ messages in thread
* Re: Switching the default for -fabi-version
@ 2003-10-22 10:41 Richard Guenther
  2003-10-22 10:55 ` Mark Mitchell
  0 siblings, 1 reply; 33+ messages in thread
From: Richard Guenther @ 2003-10-22 10:41 UTC (permalink / raw)
  To: Joe Buck; +Cc: Mark Mitchell, gcc

Joe Buck wrote:
>
>On Tue, Oct 21, 2003 at 04:24:38PM -0700, Mark Mitchell wrote:
>> Since nobody really objected to David Moore's suggestion that we make
>> -fabi-version=2 the default, I plan to change the default value of the
>> flag on the mainline in the next few days.
>>
>> If you object, speak up now!
>
>No objection, though I am still curious as to whether this actually makes
>a difference for any actual, widely used source code out there (that is,
>does -Wabi report anything for real programs that aren't specially
>constructed to tweak ABI bugs?  How often?)

Just throwing g++-3.4 with -Wabi at my POOMA based scientific code spits
out a lot of

warning: layout of classes derived from empty class
`XXX insert you favorite class here' may change in a future version of GCC

and

warning: the mangled name of `typename T::AddResult_t operator-(const
DomainBase<T1>&, const DomainDelta<Dim, C>&) [with T =
DomainTraits<Interval<2> >, int Dim = 2, int C = 2]' will change in a
future version of GCC

and

warning: `VectorEngine<2, double, BinaryVectorOp<Vector<2, double, Full>,
Vector<2, double, Full>, OpAdd> >::op_m' contains empty classes which may
cause base classes to be placed at different locations in a future version
of GCC

and

warning: vtable layout for class `DistributedMapper<2>' may not be
ABI-compliant and may change in a future version of GCC due to implicit
virtual destructor

which are not very clear (may vs. will) and omitting of a reason (in case
of the second warning). So to say, if I can still specify -fabi-version=0
I'm happy with changing. But you could try to make sure you dont break
libstdc++ compatibility with this change?

Thanks,

Richard.

--
Richard Guenther <richard dot guenther at uni-tuebingen dot de>
WWW: http://www.tat.physik.uni-tuebingen.de/~rguenth/

^ permalink raw reply	[flat|nested] 33+ messages in thread
* Re: Switching the default for -fabi-version
@ 2003-10-22 16:57 Benjamin Kosnik
  2003-10-22 17:03 ` Mark Mitchell
  0 siblings, 1 reply; 33+ messages in thread
From: Benjamin Kosnik @ 2003-10-22 16:57 UTC (permalink / raw)
  To: mark; +Cc: gcc


Mark, I think this is a great idea, and am in favor of this switch. 

I think sufficiently clever system integrators will be able to rig a
gcc-3.4 so that it is multi-abi: ie a gcc-3.4-fabi=2 with
libstdc++.so.6, and gcc-3.4-fabi=1 with libstdc++.so.5 (from the last
stable branch.) I believe this will be able to keep compatibility
with..... gcc-3.3.x.... and still allow newly-compiled code to take
advantage of the newer runtime ABI and bug-fixed compiler ABI.

best,
benjamin

^ permalink raw reply	[flat|nested] 33+ messages in thread
* Re: Switching the default for -fabi-version
@ 2003-10-22 17:17 Benjamin Kosnik
  0 siblings, 0 replies; 33+ messages in thread
From: Benjamin Kosnik @ 2003-10-22 17:17 UTC (permalink / raw)
  To: gcc; +Cc: jbuck


> Certainly I'd like to see whether -Wabi reports any warnings when
> building any of KDE (including Qt), the C++ parts of Gnome, OpenOffice,
> and Mozilla. 

It doesn't matter, but libstdc++ gets errors with -Wabi. It looks like
POOMA, and KDE do as well. 

>However, the KDE folks tell me that the first issue is
> getting 3.4 to build KDE at all; there appear to be many problems with
> stray semicolons that the new parser rejects.

Missing semicolons in member class declarations also have different
warning messages, and locations. For people who have gotten used to the
old messages, a bit of re-training is necessary.

-benjamin

^ permalink raw reply	[flat|nested] 33+ messages in thread

end of thread, other threads:[~2003-10-26 17:55 UTC | newest]

Thread overview: 33+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2003-10-22  0:49 Switching the default for -fabi-version Mark Mitchell
2003-10-22  0:53 ` Joe Buck
2003-10-22  1:58   ` Mark Mitchell
2003-10-22 14:53   ` Michael Matz
2003-10-22 15:02     ` Allan Sandfeld
2003-10-22 15:02       ` Michael Matz
2003-10-22 15:09     ` law
2003-10-22 15:20       ` Michael Matz
2003-10-22 16:33       ` Mark Mitchell
2003-10-22  2:02 ` Mike Stump
2003-10-22  2:36   ` Mark Mitchell
2003-10-22  7:08     ` law
2003-10-22  8:28       ` Mark Mitchell
2003-10-22  8:30         ` law
2003-10-22  7:49   ` Joe Buck
2003-10-22  8:09     ` Mark Mitchell
2003-10-22 10:51 ` Nathan Sidwell
2003-10-22 11:03   ` Mark Mitchell
2003-10-22 10:41 Richard Guenther
2003-10-22 10:55 ` Mark Mitchell
2003-10-22 11:32   ` Richard Guenther
2003-10-22 13:02   ` Allan Sandfeld
2003-10-22 13:57     ` Gabriel Dos Reis
2003-10-22 15:18   ` law
2003-10-22 15:23     ` Gabriel Dos Reis
2003-10-22 16:43     ` Mark Mitchell
2003-10-26 17:55   ` Florian Weimer
2003-10-26 19:10     ` Andreas Jaeger
2003-10-26 22:53       ` Florian Weimer
2003-10-27  0:46         ` Zack Weinberg
2003-10-22 16:57 Benjamin Kosnik
2003-10-22 17:03 ` Mark Mitchell
2003-10-22 17:17 Benjamin Kosnik

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).