* [PATCH] Fix PR48210
@ 2011-03-21 13:41 Richard Guenther
2011-03-21 13:45 ` Rainer Orth
0 siblings, 1 reply; 6+ messages in thread
From: Richard Guenther @ 2011-03-21 13:41 UTC (permalink / raw)
To: gcc-patches
Pretty obvious, require at least 1 partition.
Richard.
2011-03-31 Richard Guenther <rguenther@suse.de>
PR lto/48210
* params.def (lto-partitions): Require at least 1 partition.
Index: gcc/params.def
===================================================================
--- gcc/params.def (revision 171229)
+++ gcc/params.def (working copy)
@@ -868,7 +868,7 @@ DEFPARAM (PARAM_DEVIRT_TYPE_LIST_SIZE,
DEFPARAM (PARAM_LTO_PARTITIONS,
"lto-partitions",
"Number of paritions program should be split to",
- 32, 0, 0)
+ 32, 1, 0)
DEFPARAM (MIN_PARTITION_SIZE,
"lto-min-partition",
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Fix PR48210
2011-03-21 13:41 [PATCH] Fix PR48210 Richard Guenther
@ 2011-03-21 13:45 ` Rainer Orth
2011-03-21 14:18 ` Joseph S. Myers
0 siblings, 1 reply; 6+ messages in thread
From: Rainer Orth @ 2011-03-21 13:45 UTC (permalink / raw)
To: Richard Guenther; +Cc: gcc-patches
Richard Guenther <rguenther@suse.de> writes:
> Index: gcc/params.def
> ===================================================================
> --- gcc/params.def (revision 171229)
> +++ gcc/params.def (working copy)
> @@ -868,7 +868,7 @@ DEFPARAM (PARAM_DEVIRT_TYPE_LIST_SIZE,
> DEFPARAM (PARAM_LTO_PARTITIONS,
> "lto-partitions",
> "Number of paritions program should be split to",
Not your patch, but ^ typo.
Rainer
--
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Fix PR48210
2011-03-21 13:45 ` Rainer Orth
@ 2011-03-21 14:18 ` Joseph S. Myers
2011-03-21 15:29 ` Richard Guenther
0 siblings, 1 reply; 6+ messages in thread
From: Joseph S. Myers @ 2011-03-21 14:18 UTC (permalink / raw)
To: Rainer Orth; +Cc: Richard Guenther, gcc-patches
On Mon, 21 Mar 2011, Rainer Orth wrote:
> Richard Guenther <rguenther@suse.de> writes:
>
> > Index: gcc/params.def
> > ===================================================================
> > --- gcc/params.def (revision 171229)
> > +++ gcc/params.def (working copy)
> > @@ -868,7 +868,7 @@ DEFPARAM (PARAM_DEVIRT_TYPE_LIST_SIZE,
> > DEFPARAM (PARAM_LTO_PARTITIONS,
> > "lto-partitions",
> > "Number of paritions program should be split to",
>
> Not your patch, but ^ typo.
(PR 47911.)
--
Joseph S. Myers
joseph@codesourcery.com
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Fix PR48210
2011-03-21 14:18 ` Joseph S. Myers
@ 2011-03-21 15:29 ` Richard Guenther
2011-03-21 15:30 ` Rainer Orth
0 siblings, 1 reply; 6+ messages in thread
From: Richard Guenther @ 2011-03-21 15:29 UTC (permalink / raw)
To: Joseph S. Myers; +Cc: Rainer Orth, gcc-patches
On Mon, 21 Mar 2011, Joseph S. Myers wrote:
> On Mon, 21 Mar 2011, Rainer Orth wrote:
>
> > Richard Guenther <rguenther@suse.de> writes:
> >
> > > Index: gcc/params.def
> > > ===================================================================
> > > --- gcc/params.def (revision 171229)
> > > +++ gcc/params.def (working copy)
> > > @@ -868,7 +868,7 @@ DEFPARAM (PARAM_DEVIRT_TYPE_LIST_SIZE,
> > > DEFPARAM (PARAM_LTO_PARTITIONS,
> > > "lto-partitions",
> > > "Number of paritions program should be split to",
> >
> > Not your patch, but ^ typo.
>
> (PR 47911.)
Committed.
2011-03-21 Richard Guenther <rguenther@suse.de>
PR translation/47911
* params.def (lto-partitions): Fix typo.
(lto-min-partition): Fix wording.
Index: gcc/params.def
===================================================================
--- gcc/params.def (revision 171236)
+++ gcc/params.def (working copy)
@@ -867,12 +867,12 @@ DEFPARAM (PARAM_DEVIRT_TYPE_LIST_SIZE,
DEFPARAM (PARAM_LTO_PARTITIONS,
"lto-partitions",
- "Number of paritions program should be split to",
+ "Number of partitions the program should be split to",
32, 1, 0)
DEFPARAM (MIN_PARTITION_SIZE,
"lto-min-partition",
- "Size of minimal paritition for WHOPR (in estimated instructions)",
+ "Minimal size of a paritition for LTO (in estimated instructions)",
1000, 0, 0)
/* Diagnostic parameters. */
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Fix PR48210
2011-03-21 15:29 ` Richard Guenther
@ 2011-03-21 15:30 ` Rainer Orth
2011-03-21 15:34 ` Richard Guenther
0 siblings, 1 reply; 6+ messages in thread
From: Rainer Orth @ 2011-03-21 15:30 UTC (permalink / raw)
To: Richard Guenther; +Cc: Joseph S. Myers, gcc-patches
Richard Guenther <rguenther@suse.de> writes:
> DEFPARAM (MIN_PARTITION_SIZE,
> "lto-min-partition",
> - "Size of minimal paritition for WHOPR (in estimated instructions)",
> + "Minimal size of a paritition for LTO (in estimated instructions)",
Another typo :-) ^
Rainer
--
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Fix PR48210
2011-03-21 15:30 ` Rainer Orth
@ 2011-03-21 15:34 ` Richard Guenther
0 siblings, 0 replies; 6+ messages in thread
From: Richard Guenther @ 2011-03-21 15:34 UTC (permalink / raw)
To: Rainer Orth; +Cc: Joseph S. Myers, gcc-patches
On Mon, 21 Mar 2011, Rainer Orth wrote:
> Richard Guenther <rguenther@suse.de> writes:
>
> > DEFPARAM (MIN_PARTITION_SIZE,
> > "lto-min-partition",
> > - "Size of minimal paritition for WHOPR (in estimated instructions)",
> > + "Minimal size of a paritition for LTO (in estimated instructions)",
>
> Another typo :-) ^
Fixed.
Clearly shows the value of spell-checkers in IDEs ;) Of course I
simply blame Honza ... ;)
Richard.
2011-03-21 Richard Guenther <rguenther@suse.de>
* params.def (lto-min-partition): Fix typo.
Index: gcc/params.def
===================================================================
--- gcc/params.def (revision 171244)
+++ gcc/params.def (working copy)
@@ -872,7 +872,7 @@ DEFPARAM (PARAM_LTO_PARTITIONS,
DEFPARAM (MIN_PARTITION_SIZE,
"lto-min-partition",
- "Minimal size of a paritition for LTO (in estimated instructions)",
+ "Minimal size of a partition for LTO (in estimated instructions)",
1000, 0, 0)
/* Diagnostic parameters. */
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2011-03-21 15:34 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-03-21 13:41 [PATCH] Fix PR48210 Richard Guenther
2011-03-21 13:45 ` Rainer Orth
2011-03-21 14:18 ` Joseph S. Myers
2011-03-21 15:29 ` Richard Guenther
2011-03-21 15:30 ` Rainer Orth
2011-03-21 15:34 ` Richard Guenther
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).