Patchwork Fix PR48210

login
register
mail settings
Submitter Richard Guenther
Date March 21, 2011, 1:41 p.m.
Message ID <alpine.LNX.2.00.1103211440520.810@zhemvz.fhfr.qr>
Download mbox | patch
Permalink /patch/87781/
State New
Headers show

Comments

Richard Guenther - March 21, 2011, 1:41 p.m.
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.
Rainer Orth - March 21, 2011, 1:44 p.m.
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
Joseph S. Myers - March 21, 2011, 2:18 p.m.
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.)

Patch

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",