RFR: 8373999: C2: apply KnownBits and unsigned bounds to Add / Sub operations [v6]

Quan Anh Mai qamai at openjdk.org
Wed Jan 28 08:19:20 UTC 2026


> Hi,
> 
> This PR improves the implementation of `AddNode/SubNode::Value` by taking advantage of the additional information in `TypeInt`. The implementation has some pretty non-trivial logic. Fortunately, the test infrastructure is already there.
> 
> Please take a look and leave your reviews, thanks a lot.

Quan Anh Mai has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains eight additional commits since the last revision:

 - Fix merge conflict, address review
 - Merge branch 'master' into addsub
 - Improve comments
 - copyright year
 - Merge branch 'master' into addsub
 - Merge branch 'master' into addsub
 - include order
 - Improve Add/SubNode::Value with unsigned bounds and known bits

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/28897/files
  - new: https://git.openjdk.org/jdk/pull/28897/files/ae17b24e..1b3ea38d

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=28897&range=05
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=28897&range=04-05

  Stats: 81638 lines in 2598 files changed: 39493 ins; 14781 del; 27364 mod
  Patch: https://git.openjdk.org/jdk/pull/28897.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/28897/head:pull/28897

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


More information about the hotspot-compiler-dev mailing list