[patch i386]: PR/39356 name collision of __chkstk

[Kai Tietz]

Hi Aaron,

2009/3/25 Aaron W. LaFramboise <aaronavay62@aaronwl.com>:
> Hi Kai,
>
> Kai Tietz wrote:
>
>> 2009-03-19  Kai Tietz  <kai.tietz@onevision.com>
>>
>>        * config/i386/i386.md (allocate_stack_worker_32): Use
>>        ___gnu_chkstk.
>>        (allocate_stack_worker_64): Likewise.
>>        * config/i386/cygwin.asm  (___chkstk): Rename this
>>        symbol for 64-bit to ___gnu_chkstk and add it for 32-bit
>>        as alias for ___chkstk.
>
> Can you clarify that the end result of this results in link compatibility
> with MSVC on x86_64, including MSVC code that emits _*chkstk calls?

Yes, by this patch (for x64) it works.

> My understanding is that the VC chkstk does not do as much stuff as GCC
> needs the probe to do, so you're making a new one, and it was just a mistake
> that it happened to be named the same thing as the MSVC one.  Is this
> correct?

Right.

> (I have no opinion on this issue otherwise.  I'm just trying to understand
> whats going on.)
>

Cheers,
Kai
-- 
|  (\_/) This is Bunny. Copy and paste
| (='.'=) Bunny into your signature to help
| (")_(") him gain world domination