Re: RFR: 8339113: AccessFlags can be u2 in metadata [v11]

2025-01-06 Thread Vladimir Ivanov
On Mon, 6 Jan 2025 13:47:35 GMT, Coleen Phillimore wrote: >> Yes, we are. > > I had a little experiment. > > extern "C" int printf(const char *,...); > int main() { > unsigned short ss = 0x8000; > printf("does unsigned sign extend to int? %d\n", int(ss)); > } IMO you could just call `as_int

Re: RFR: 8339113: AccessFlags can be u2 in metadata [v11]

2025-01-06 Thread Coleen Phillimore
On Mon, 6 Jan 2025 13:44:27 GMT, Coleen Phillimore wrote: >> src/hotspot/share/ci/ciFlags.cpp line 95: >> >>> 93: // ciFlags::print >>> 94: void ciFlags::print(outputStream* st) { >>> 95: st->print(" flags=%x", _flags.as_unsigned_short()); >> >> Here, and elsewhere, are we relying on an impli

Re: RFR: 8339113: AccessFlags can be u2 in metadata [v11]

2025-01-06 Thread Coleen Phillimore
On Mon, 6 Jan 2025 03:47:05 GMT, David Holmes wrote: >> Coleen Phillimore has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Fix more copyrights. > > src/hotspot/share/ci/ciFlags.hpp line 71: > >> 69: >> 70: // Conversion >> 71: jint

Re: RFR: 8339113: AccessFlags can be u2 in metadata [v11]

2025-01-06 Thread Coleen Phillimore
On Mon, 6 Jan 2025 03:44:09 GMT, David Holmes wrote: >> Coleen Phillimore has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Fix more copyrights. > > src/hotspot/share/ci/ciFlags.cpp line 95: > >> 93: // ciFlags::print >> 94: void ciFlags:

Re: RFR: 8339113: AccessFlags can be u2 in metadata [v11]

2025-01-05 Thread David Holmes
On Fri, 3 Jan 2025 22:00:34 GMT, Coleen Phillimore wrote: >> Please review this change that makes AccessFlags and modifier_flags u2 types >> and removes the last remnants of Hotspot adding internal access flags. This >> change moves AccessFlags and modifier_flags in Klass to alignment gaps >>

Re: RFR: 8339113: AccessFlags can be u2 in metadata [v11]

2025-01-03 Thread Coleen Phillimore
> Please review this change that makes AccessFlags and modifier_flags u2 types > and removes the last remnants of Hotspot adding internal access flags. This > change moves AccessFlags and modifier_flags in Klass to alignment gaps saving > 16 bytes. From pahole: so it's a bit better. > > befor