diff mbox

PATCH to change -fabi-version default to 0

Message ID 5399FCE6.7080000@redhat.com
State New
Headers show

Commit Message

Jason Merrill June 12, 2014, 7:17 p.m. UTC 
I talked about doing this in 4.9 
(https://gcc.gnu.org/ml/gcc/2013-03/msg00008.html), but decided to put 
it off along with the libstdc++ ABI transition.  I think it's time now.

Tested x86_64-pc-linux-gnu, applying to trunk.

Comments

Mike Stump June 12, 2014, 7:36 p.m. UTC | #1 
On Jun 12, 2014, at 12:17 PM, Jason Merrill <jason@redhat.com> wrote:
> I talked about doing this in 4.9 (https://gcc.gnu.org/ml/gcc/2013-03/msg00008.html), but decided to put it off along with the libstdc++ ABI transition.  I think it's time now.

Is a doc change needed?

> @opindex fabi-version
> Use version @var{n} of the C++ ABI@.  The default is version 2.
Jason Merrill June 12, 2014, 9:03 p.m. UTC | #2 
On 06/12/2014 03:36 PM, Mike Stump wrote:
> On Jun 12, 2014, at 12:17 PM, Jason Merrill <jason@redhat.com> wrote:
>> I talked about doing this in 4.9 (https://gcc.gnu.org/ml/gcc/2013-03/msg00008.html), but decided to put it off along with the libstdc++ ABI transition.  I think it's time now.
>
> Is a doc change needed?

Yep, I updated the docs in the -Wabi=n patch.

Jason
diff mbox

Patch

commit a2aa0efcd1f27e85a4c652f5177c66686f530a96
Author: Jason Merrill <jason@redhat.com>
Date:   Mon Jun 9 16:37:43 2014 -0400

    	* common.opt (fabi-version): Change default to 0.

diff --git a/gcc/common.opt b/gcc/common.opt
index f61fab5..7f05092 100644
--- a/gcc/common.opt
+++ b/gcc/common.opt
@@ -808,7 +808,8 @@  Driver Undocumented
 ; Additional positive integers will be assigned as new versions of
 ; the ABI become the default version of the ABI.
 fabi-version=
-Common Joined RejectNegative UInteger Var(flag_abi_version) Init(2)
+Common Joined RejectNegative UInteger Var(flag_abi_version) Init(0)
+The version of the C++ ABI in use
 
 faggressive-loop-optimizations
 Common Report Var(flag_aggressive_loop_optimizations) Optimization Init(1)