RFR: 8373591: C2: Fix the memory around some intrinsics nodes
Quan Anh Mai
qamai at openjdk.org
Fri Dec 12 12:51:40 UTC 2025
Hi,
This is extracted from #28570 , there are 2 issues here:
- Some intrinsics nodes advertise incorrect `adr_type`. For example, `AryEqNode` reports `adr_type` being `TypeAryPtr::BYTES` (it inherits this from `StrIntrinsicNode`). This is incorrect, however, as it can accept `char[]` inputs, too. Another case is `VectorizedHashCodeNode`, which reports its `adr_type` being `TypePtr::BOTTOM`, but it actually extracts a memory slice and does not consume the whole memory.
- For nodes such as `StrInflatedCopyNode`, as they consume more than they produce, during scheduling, we need to compute anti-dependencies. This is not the case, so we should fix it by making the nodes kill all the memory they consume. This issue is often not present because these intrinsics are not exposed bare to general usage.
Please kindly review, thanks a lot.
-------------
Commit messages:
- Fix memory around intrinsics nodes
Changes: https://git.openjdk.org/jdk/pull/28789/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=28789&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8373591
Stats: 274 lines in 6 files changed: 177 ins; 5 del; 92 mod
Patch: https://git.openjdk.org/jdk/pull/28789.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/28789/head:pull/28789
PR: https://git.openjdk.org/jdk/pull/28789
More information about the hotspot-compiler-dev
mailing list