Re: RFR: 8256052: Remove unused allocation type from fieldInfo [v4]

2020-11-10 Thread Frederic Parain
On Mon, 9 Nov 2020 20:23:55 GMT, Harold Seigel wrote: >> Frederic Parain has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Fix comment > > Looks good! Claes, Lois, Harold, Thank you for your reviews and feedback. Fred - PR

Re: RFR: 8256052: Remove unused allocation type from fieldInfo [v4]

2020-11-09 Thread Harold Seigel
On Mon, 9 Nov 2020 20:05:09 GMT, Frederic Parain 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 yo

Re: RFR: 8256052: Remove unused allocation type from fieldInfo [v4]

2020-11-09 Thread Frederic Parain
> 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 increm