7u40 RFR: 8012144 - Missing memory barriers

Lindenmaier, Goetz goetz.lindenmaier at sap.com
Wed Jul 10 08:39:11 PDT 2013


that's fine, 

Best regards,

-----Original Message-----
From: Aleksey Shipilev [mailto:aleksey.shipilev at oracle.com] 
Sent: Mittwoch, 10. Juli 2013 14:36
To: David Holmes
Cc: Lindenmaier, Goetz; hotspot-dev developers
Subject: Re: 7u40 RFR: 8012144 - Missing memory barriers

On 07/10/2013 05:07 AM, David Holmes wrote:
> On 9/07/2013 7:01 PM, Lindenmaier, Goetz wrote:
>> Actually, I would use X86 instead of IA32 || AMD64.
>> (maybe add #include utilities/macros.hpp, but X86 is
>> defined before all uses of taskqueue.hpp. I tested this
>> on linux.)
> Unfortunately I don't have time to make this change and re-do the
> testing that has already been done.

Actually, there is a few uses like that already. Hence, it makes little
sense to fix this nit in current patch, but rather, do the rehash of
IA32||AMD64 -> X86 macro separately.

$ ack-grep -Q "defined(IA32) || defined(AMD64)" hotspot/src/
2294:#if defined(IA32) || defined(AMD64)

97:#if defined(IA32) || defined(AMD64)

305:#if defined(IA32) || defined(AMD64)

139:#if defined(IA32) || defined(AMD64) || defined(ARM)

1193:#if defined(IA32) || defined(AMD64) || defined(ARM)

2989:#if defined(IA32) || defined(AMD64)
3000:#if defined(IA32) || defined(AMD64)

1412:#if defined(IA32) || defined(AMD64)


More information about the hotspot-dev mailing list