RFR: 8317132: Prepare HotSpot for permissive- [v5]

[David Holmes]

On Tue, 3 Oct 2023 02:22:11 GMT, Julian Waters <jwaters at openjdk.org> wrote:

>> Right, so we're employing a workaround to silence the warning. That seems like the tail wagging the dog though. 
>> 
>> There's a potential issue in this code that the warning tries to point out: the code after the label might access the uninitialized variables, since they are in scope. Re-writing the code in the way you did makes the warning go away, but not the potential issue?
>
> Not entirely sure what the issue is, since the code that goto jumps to never uses the uninitialized locals. I don't really have any other solutions to solve this issue unfortunately :/ Maybe @djelinski can add some perspective on this?

I think killing the goto makes most sense here.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15955#discussion_r1343649325