RFR: 8256052: Remove unused allocation type from fieldInfo [v3]

Harold Seigel hseigel at openjdk.java.net
Mon Nov 9 19:58:15 UTC 2020


On Mon, 9 Nov 2020 19:54:58 GMT, Frederic Parain <fparain at 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


More information about the serviceability-dev mailing list