RFR: 8293170: Improve encoding of the debuginfo nmethod section

Mon Oct 3 20:24:53 UTC 2022

On Mon, 3 Oct 2022 15:05:24 GMT, Boris Ulasevich <bulasevich at openjdk.org> wrote:

>> My builds failed:
>> 
>> src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/code/CompressedSparceDataReadStream.java:42: warning: [lossy-conversions] implicit cast from int to byte in compound assignment is possibly lossy
>>        b |= (0xFF & curr_byte_) >> (8 - byte_pos_);
>>                                 ^
>
>> My builds failed:
>> 
>> ```
>> src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/code/CompressedSparceDataReadStream.java:42: warning: [lossy-conversions] implicit cast from int to byte in compound assignment is possibly lossy
>>        b |= (0xFF & curr_byte_) >> (8 - byte_pos_);
>>                                 ^
>> ```
> 
> Yes. It was a collision with "8244681: Add a warning for possibly lossy conversion in compound assignments" change. I rebased my branch and fixed the issue. Now it should be Ok.

@bulasevich, it would be useful to have some examples of cases either in the JBS issues or the PR.

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

PR: https://git.openjdk.org/jdk/pull/10025