diff mbox

Fix PR48762

Message ID 20130409111349.GA14821@redhat.com
State New
Headers show

Commit Message

Marek Polacek April 9, 2013, 11:13 a.m. UTC 
On Tue, Apr 09, 2013 at 11:45:04AM +0200, Eric Botcazou wrote:
> > Alternatively, we can bump the minimum of that param, as usual ;)
> 
> Let's do that and bump it to 1, my understanding is that 0 and 1 are 
> equivalent for this param.

Alright.  So ok to apply this one (trunk/4.8)?

2013-04-09  Marek Polacek  <polacek@redhat.com>

	PR tree-optimization/48762
	* params.def (PARAM_MAX_CSE_INSNS): Increase the minimum
	value to 1.


	Marek

Comments

Jeff Law April 9, 2013, 12:56 p.m. UTC | #1 
On 04/09/2013 05:13 AM, Marek Polacek wrote:
> On Tue, Apr 09, 2013 at 11:45:04AM +0200, Eric Botcazou wrote:
>>> Alternatively, we can bump the minimum of that param, as usual ;)
>>
>> Let's do that and bump it to 1, my understanding is that 0 and 1 are
>> equivalent for this param.
>
> Alright.  So ok to apply this one (trunk/4.8)?
>
> 2013-04-09  Marek Polacek  <polacek@redhat.com>
>
> 	PR tree-optimization/48762
> 	* params.def (PARAM_MAX_CSE_INSNS): Increase the minimum
> 	value to 1.
OK.
Jeff
diff mbox

Patch

--- gcc/params.def.mp	2013-04-09 12:52:07.838278693 +0200
+++ gcc/params.def	2013-04-09 12:52:28.705349278 +0200
@@ -451,7 +451,7 @@  DEFPARAM(PARAM_MAX_GOTO_DUPLICATION_INSN
 DEFPARAM(PARAM_MAX_CSE_PATH_LENGTH,
 	 "max-cse-path-length",
 	 "The maximum length of path considered in cse",
-	 10, 0, 0)
+	 10, 1, 0)
 DEFPARAM(PARAM_MAX_CSE_INSNS,
 	 "max-cse-insns",
 	 "The maximum instructions CSE process before flushing",