RFR: JDK-8209118: Abstract SATBMarkQueueSet's ThreadLocalData access

Roman Kennke rkennke at redhat.com
Thu Aug 9 18:22:38 UTC 2018


Duh. Thanks for checking this. Don't know what happened to the new files. Will add the missing include. I suppose that does not require extra review?

Thanks, Roman


Am 9. August 2018 20:08:25 MESZ schrieb Kim Barrett <kim.barrett at oracle.com>:
>> On Aug 9, 2018, at 5:59 AM, Roman Kennke <rkennke at redhat.com> wrote:
>> 
>> 2018-08-09-0900164.roman.source
>
>New file g1SATBMarkQueueSet.cpp is missing #include “precompiled.hpp”.
>
>Sorry I missed that when reviewing.
>
>Strangely, the new files aren’t in the webrev patch file
>(jdk-jdk.changeset), even though
>they show up in the file list. I don’t understand what is going on
>here.  I found this when
>I was going to apply the patch to a local repo, fix the missing
>include, and run tests to
>see if there were any other problems that were masked by the missing
>include, to save
>some time iterating between us.

-- 
Diese Nachricht wurde von meinem Android-Gerät mit K-9 Mail gesendet.



More information about the hotspot-gc-dev mailing list