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. -- Steve 20170425 https://www.youtube.com/watch?v=VWUpyCsUKR4 20161221 https://www.youtube.com/watch?v=IbCHE-hONow