[patch, fortran] Fix PR 82567

Mikael Morin morin-mikael@orange.fr
Wed Oct 18 12:03:00 GMT 2017


Le 18/10/2017 à 04:05, Steve Kargl a écrit :
> On Tue, Oct 17, 2017 at 06:14:16PM -0700, Jerry DeLisle wrote:
>> On 10/17/2017 03:36 PM, Thomas Koenig wrote:
>>> Hello world,
>>>
>>> this patch fixes a regression with long compile times,
>>> which came about due to our handling of array constructors
>>> at compile time.  This, togeteher with a simplification in
>>> front end optimization, led to long compile times and large
>>> code.
>>>
>>> Regression-tested. OK for trunk and the other affected branches?
>>>
>>
>> Well I know 42 is the answer to the ultimate question of the universe so this
>> must be OK.  I just don't know what the question is.
>>
>> OK and thanks,
>>
>> Jerry
>>
>> +#define CONSTR_LEN_MAX 42
> 
> Actually, I was wondering about the choice myself.  With
> most common hardware having fairly robust L1 and L2 cache
> sizes, a double precision array constructor with 42
> elements only occupies 336 bytes.  Seems small.
> 
There is a -fmax-array-constructor=n option. Can’t we use it for the limit?



More information about the Gcc-patches mailing list