RFR: 8226525: HotSpot compile-time error for x86-32
Kim Barrett
kim.barrett at oracle.com
Mon Jul 1 17:49:07 UTC 2019
> On Jul 1, 2019, at 10:46 AM, Aleksey Shipilev <shade at redhat.com> wrote:
>
> On 7/1/19 4:22 PM, Andrew Haley wrote:
>> This asm statement:
>>
>> asm__ volatile ("cpuid " : "+a" (idx) : : "ebx", "ecx", "edx", "memory")
>>
>> ... breaks on 32-bit systems because the GCC we use doesn't allow EBX to be
>> clobbered. Fixed thusly:
>>
>> http://cr.openjdk.java.net/~aph/8226525/
>
> Looks okay to me. Put the comment, e.g.:
> // EBX is a reserved register on 32-bit Linux systems, cannot clobber it.
>
> -Aleksey
Looks good. +1 on the additional comment.
More information about the hotspot-dev
mailing list