RFR: 8350896: Integer/Long.compress gets wrong type from CompressBitsNode::Value

Jatin Bhateja jbhateja at openjdk.org
Fri Mar 7 17:42:07 UTC 2025


Hi All,

This bugfix patch fixes incorrect value computation for Integer/Long. compress APIs.

Problems occur with a constant input and variable mask where the input's value is equal to the lower bound of the mask value., In this case, an erroneous value range estimation results in a constant value. Existing value routine first attempts to constant fold the compression operation if both input and compression mask are constant values; otherwise, it attempts to constrain the value range of result based on the upper and lower bounds of mask type.

New IR test covers the issue reported in the bug report along with a case for value range based logic pruning.

Kindly review and share your feedback.

Best Regards,
Jatin

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

Commit messages:
 - 8350896: Integer/Long.compress gets wrong type from CompressBitsNode::Value

Changes: https://git.openjdk.org/jdk/pull/23947/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=23947&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8350896
  Stats: 163 lines in 3 files changed: 145 ins; 15 del; 3 mod
  Patch: https://git.openjdk.org/jdk/pull/23947.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/23947/head:pull/23947

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


More information about the hotspot-compiler-dev mailing list