RFR: 8346664: C2: Optimize mask check with constant offset [v3]

Matthias Ernst duke at openjdk.org
Mon Jan 27 08:37:07 UTC 2025


On Mon, 27 Jan 2025 07:49:27 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> What's left to do here? Thanks!
>
> @mernst-github Thanks for the test update!
> 
> I see there is still discussion around this line:
> `  return zeros > 0 && ((((jlong)1) << zeros) > mask_t->hi_as_long() && mask_t->lo_as_long() >= 0);`
> 
> It is indeed difficult to read. Maybe even just some clever names of temporary variables could help. But I leave that up to you ;)
> 
> Ping me when you want me to run testing and review again :)

@eme64 I do think the new version is much clearer. I've committed it, and I think this PR is good to go.

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

PR Comment: https://git.openjdk.org/jdk/pull/22856#issuecomment-2615125714


More information about the hotspot-compiler-dev mailing list