Patchwork Remove old help texts from ignored c.opt options

login
register
mail settings
Submitter Joseph S. Myers
Date Sept. 2, 2010, 4:35 p.m.
Message ID <Pine.LNX.4.64.1009021635150.4134@digraph.polyomino.org.uk>
Download mbox | patch
Permalink /patch/63508/
State New
Headers show

Comments

Joseph S. Myers - Sept. 2, 2010, 4:35 p.m.
While marking options with Ignore I noticed that various such c.opt
options still had help texts describing what the semantics of those
options once were.  Such texts are useless and there's no point in
having them present or having translators translate them; I've changed
them to "No longer supported".  I haven't tried to give greater
consistency to which options have a help text at all versus no text
(with or without being marked Undocumented), or to work out if there
is any logic to which options give warnings and which are quietly
ignored; I think that's really a matter for the C++ maintainers.
(Logically, a non-semantic option that is no longer implemented should
be quietly ignored - thus various old optimizer options are quietly
ignored - while one affecting the semantics of code more reasonably
receives a warning or error.)

Bootstrapped with no regressions on x86_64-unknown-linux-gnu.  Applied
to mainline.

2010-09-02  Joseph Myers  <joseph@codesourcery.com>

	* c.opt (falt-external-templates, fhuge-objects, fvtable-gc,
	fvtable-thunks, fxref): Mark no longer supported in help text.

Patch

Index: c.opt
===================================================================
--- c.opt	(revision 163771)
+++ c.opt	(working copy)
@@ -538,7 +538,7 @@ 
 
 falt-external-templates
 C++ ObjC++ Ignore Warn(switch %qs is no longer supported)
-Change when template instances are emitted
+No longer supported
 
 fasm
 C ObjC C++ ObjC++ Var(flag_no_asm, 0)
@@ -644,7 +644,7 @@ 
 
 fhuge-objects
 C++ ObjC++ Ignore Warn(switch %qs is no longer supported)
-Enable support for huge objects
+No longer supported
 
 fimplement-inlines
 C++ ObjC++ Var(flag_implement_inlines) Init(1)
@@ -843,11 +843,11 @@ 
 
 fvtable-gc
 C++ ObjC++ Ignore Warn(switch %qs is no longer supported)
-Discard unused virtual functions
+No longer supported
 
 fvtable-thunks
 C++ ObjC++ Ignore Warn(switch %qs is no longer supported)
-Implement vtables using thunks
+No longer supported
 
 fweak
 C++ ObjC++ Var(flag_weak) Init(1)
@@ -863,7 +863,7 @@ 
 
 fxref
 C++ ObjC++ Ignore Warn(switch %qs is no longer supported)
-Emit cross referencing information
+No longer supported
 
 fzero-link
 ObjC ObjC++ Var(flag_zero_link)