[m-rev.] for review: setting the reserve stack size on Windows
Julien Fischer
jfischer at opturion.com
Wed Apr 10 14:35:07 AEST 2013
On Wed, Apr 10, 2013 at 2:23 PM, Paul Bone <paul at bone.id.au> wrote:
> On Wed, Apr 10, 2013 at 01:36:28PM +1000, Julien Fischer wrote:
>> For review by anyone.
>>
>> Add an option for controlling the C stack size on Windows.
>>
>> Add a new option, --cstack-reserve-size, that specifies the amount of virtual
>> memory that should be reserved for the stack in executables on Windows systems.
>> This is useful for non-trivial programs that use the high-level C backend since
>> they often bump into the default 1Mb limit.
>>
>> compiler/options.m::
>> Add the new option.
>>
>> compiler/compile_target_code.m:
>> Pass the appropriate flags to the linker if the new option is given.
>>
>> NEWS:
>> Announce the new option.
>>
>
> I see that we don't override the default. This means that a such a program
> will crash anyway, and the user still has to go and find this option. I
> think that it is reasonable to also increase the default C stack size so that
> it is less likely that a user will have this error.
Wait until this change has bootstrapped ....
> The default value should
> also appear in the help message and users' guide.
I think that depends on the version of Windows you are using.
> You did not mention the changes to the users guide in the summary.
Fixed.
Cheers,
Julien.
More information about the reviews
mailing list