RFR: 8373591: C2: Fix the memory around some intrinsics nodes [v3]

Quan Anh Mai qamai at openjdk.org
Tue Dec 16 06:23:12 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.

Quan Anh Mai has updated the pull request incrementally with one additional commit since the last revision:

  Use MemBar instead of widening the intrinsic memory

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/28789/files
  - new: https://git.openjdk.org/jdk/pull/28789/files/1e026354..9649a2f2

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=28789&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=28789&range=01-02

  Stats: 62 lines in 3 files changed: 39 ins; 8 del; 15 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 shenandoah-dev mailing list