RFR: 8353359: C2: Or(I|L)Node::Ideal is missing AddNode::Ideal call

Hannes Greule hgreule at openjdk.org
Tue Apr 1 06:27:49 UTC 2025


Hi,

this simple change adds a missing AddNode::Ideal call to Or(I|L)Node::Ideal. See the added tests for examples of optimizations that don't apply without this change.

Please let me know what you think.

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

Commit messages:
 - Call AddNode::Ideal in Or(I|L)Node::Ideal
 - Test AddNode::Ideal optimizations for Or(I|L)

Changes: https://git.openjdk.org/jdk/pull/24348/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=24348&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8353359
  Stats: 37 lines in 3 files changed: 33 ins; 0 del; 4 mod
  Patch: https://git.openjdk.org/jdk/pull/24348.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/24348/head:pull/24348

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


More information about the hotspot-compiler-dev mailing list