RFR (XS): Move implementation of process_grey_object to concurrentMark.inline.hpp
Kim Barrett
kim.barrett at oracle.com
Thu Aug 13 13:52:38 UTC 2015
On Aug 13, 2015, at 5:55 AM, Siebenborn, Axel <axel.siebenborn at sap.com> wrote:
>
> On 12.08.2015, at 21:57, Kim Barrett wrote:
>> (If Axel has time, I'm mildly curious whether just
>> removing the inline qualifier also works around the problem.)
> We were curious too and indeed, removing the inline qualifier would work around the problem.
Interesting. I may spend some time this afternoon re-reading bits of the standard,
but I think you should report this as a bug to your compiler vendor.
>> I have one optional change to suggest, which is to move the inline
>> definition of scan_object from the .hpp file to the .inline.hpp file.
>> I think putting it in the .hpp file was a mistake on my part. Axel,
>> let me know whether you want to make that additional change.
> Yes, there is the new webrev:
>
> http://cr.openjdk.java.net/~asiebenborn/8133121_0/webrev/
Looks good.
> Thanks for reviewing and sponsoring,
> Axel
No problem. I’m guessing Volker likes this version too, but I’d like an email ACK before pushing.
More information about the hotspot-gc-dev
mailing list