Skip to content

Commit 3eedb37

Browse files
[GR-39157] Revert introduction of PerformanceIssueBailout.
PullRequest: graal/11998
2 parents 5ff0b5e + 684151f commit 3eedb37

File tree

4 files changed

+2
-47
lines changed

4 files changed

+2
-47
lines changed

compiler/src/org.graalvm.compiler.core.common/src/org/graalvm/compiler/core/common/PermanentBailoutException.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,9 +24,7 @@
2424
*/
2525
package org.graalvm.compiler.core.common;
2626

27-
import org.graalvm.compiler.debug.PerformanceIssueBailout;
28-
29-
public class PermanentBailoutException extends GraalBailoutException implements PerformanceIssueBailout {
27+
public class PermanentBailoutException extends GraalBailoutException {
3028

3129
private static final long serialVersionUID = -2683649650135362549L;
3230

compiler/src/org.graalvm.compiler.debug/src/org/graalvm/compiler/debug/DebugConfigImpl.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -242,8 +242,7 @@ private static void add(StringBuilder sb, String name, Object filter) {
242242
public RuntimeException interceptException(DebugContext debug, Throwable e) {
243243
if (e instanceof BailoutException) {
244244
boolean causedByCompilerAssert = e instanceof CausableByCompilerAssert && ((CausableByCompilerAssert) e).isCausedByCompilerAssert();
245-
boolean isPerformanceIssue = e instanceof PerformanceIssueBailout && ((PerformanceIssueBailout) e).isPerformanceIssue();
246-
if (!DebugOptions.InterceptBailout.getValue(options) && !causedByCompilerAssert && !isPerformanceIssue) {
245+
if (!DebugOptions.InterceptBailout.getValue(options) && !causedByCompilerAssert) {
247246
return null;
248247
}
249248
}

compiler/src/org.graalvm.compiler.debug/src/org/graalvm/compiler/debug/PerformanceIssueBailout.java

Lines changed: 0 additions & 35 deletions
This file was deleted.

compiler/src/org.graalvm.compiler.lir/src/org/graalvm/compiler/lir/BailoutAndRestartBackendException.java

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -40,11 +40,4 @@ public BailoutAndRestartBackendException(String msg) {
4040
public BailoutAndRestartBackendException(Throwable cause, String msg) {
4141
super(cause, msg);
4242
}
43-
44-
@Override
45-
public boolean isPerformanceIssue() {
46-
// we are going to retry compilation and throw an error if it doesn't work
47-
// see org.graalvm.compiler.core.gen.LIRCompilerBackend#emitLIR
48-
return false;
49-
}
5043
}

0 commit comments

Comments
 (0)