[GR-59402] Adjustments for graph builder plugins

PullRequest: graal/19263

Author: gergo-

compiler/src/jdk.graal.compiler/src/jdk/graal/compiler/java/BytecodeParser.java

@@ -2456,6 +2456,11 @@ public void close() {
         }
     }
 
+    @Override
+    public boolean canInvokeFallback() {
+        return true;
+    }
+
     @Override
     public Invoke invokeFallback(FixedWithNextNode predecessor, EndNode end) {
         assert isParsingInvocationPlugin();

compiler/src/jdk.graal.compiler/src/jdk/graal/compiler/nodes/calc/UnsignedDivNode.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2011, 2022, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2011, 2024, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -27,15 +27,14 @@
 import jdk.graal.compiler.core.common.type.IntegerStamp;
 import jdk.graal.compiler.core.common.type.Stamp;
 import jdk.graal.compiler.graph.NodeClass;
+import jdk.graal.compiler.nodeinfo.NodeInfo;
 import jdk.graal.compiler.nodes.ConstantNode;
 import jdk.graal.compiler.nodes.NodeView;
 import jdk.graal.compiler.nodes.ValueNode;
 import jdk.graal.compiler.nodes.extended.GuardingNode;
 import jdk.graal.compiler.nodes.spi.CanonicalizerTool;
 import jdk.graal.compiler.nodes.spi.LIRLowerable;
 import jdk.graal.compiler.nodes.spi.NodeLIRBuilderTool;
-import jdk.graal.compiler.nodeinfo.NodeInfo;
-
 import jdk.vm.ci.code.CodeUtil;
 
 @NodeInfo(shortName = "|/|")
@@ -64,6 +63,9 @@ public ValueNode canonical(CanonicalizerTool tool, ValueNode forX, ValueNode for
 
     @SuppressWarnings("unused")
     private static ValueNode canonical(UnsignedDivNode self, ValueNode forX, ValueNode forY, GuardingNode zeroCheck, Stamp stamp, NodeView view) {
+        if (!(stamp instanceof IntegerStamp)) {
+            return self != null ? self : new UnsignedDivNode(forX, forY, zeroCheck);
+        }
         int bits = ((IntegerStamp) stamp).getBits();
         if (forX.isConstant() && forY.isConstant()) {
             long yConst = CodeUtil.zeroExtend(forY.asJavaConstant().asLong(), bits);

compiler/src/jdk.graal.compiler/src/jdk/graal/compiler/nodes/graphbuilderconf/GraphBuilderContext.java

@@ -472,6 +472,14 @@ default boolean allowDeoptInPlugins() {
         return !StrictDeoptInsertionChecks.getValue(getOptions());
     }
 
+    /**
+     * Returns {@code true} if {@link #invokeFallback} can be called without throwing an
+     * unconditional error.
+     */
+    default boolean canInvokeFallback() {
+        return false;
+    }
+
     @SuppressWarnings("all")
     default Invoke invokeFallback(FixedWithNextNode predecessor, EndNode end) {
         throw new GraalError("Cannot be called on a " + getClass().getName() + " object");

compiler/src/jdk.graal.compiler/src/jdk/graal/compiler/replacements/IntrinsicGraphBuilder.java

@@ -393,6 +393,11 @@ public boolean isParsingInvocationPlugin() {
         return true;
     }
 
+    @Override
+    public boolean canInvokeFallback() {
+        return true;
+    }
+
     @Override
     public Invoke invokeFallback(FixedWithNextNode predecessor, EndNode end) {
         assert isParsingInvocationPlugin();