On Mon, 9 Nov 2020 19:54:58 GMT, Frederic Parain <fpar...@openjdk.org> wrote:

>> Please review this small cleanup code, removing the now unused allocation 
>> type from the fieldInfo structure.
>> 
>> Tested with Mach5, tiers 1 to 3 and locally by running 
>> test/hotspot/jtreg/serviceability/sa tests.
>> 
>> Thank you,
>> 
>> Fred
>
> Frederic Parain has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Remove unused symbol from vmStruct

src/hotspot/share/oops/fieldInfo.hpp line 61:

> 59:   //    [------------------offset----------------]01  - real field offset
> 60: 
> 61:   // Bit O indicates if the packed field contains an offset (O=1) or not 
> (O=1)

Hi Fred, should this comment say  "... or not (0=0)   ?

-------------

PR: https://git.openjdk.java.net/jdk/pull/1130

Reply via email to