Skip to content

Commit e868b09

Browse files
committed
add "this-escape" suppressions (JDK-8015831)
1 parent 88fd859 commit e868b09

File tree

219 files changed

+254
-43
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

219 files changed

+254
-43
lines changed

compiler/src/org.graalvm.compiler.api.test/src/org/graalvm/compiler/api/test/ExportingClassLoader.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,10 +29,12 @@
2929
* in the unnamed module associated with the loader.
3030
*/
3131
public class ExportingClassLoader extends ClassLoader {
32+
@SuppressWarnings("this-escape")
3233
public ExportingClassLoader() {
3334
ModuleSupport.exportAllPackagesTo(getClass(), this);
3435
}
3536

37+
@SuppressWarnings("this-escape")
3638
public ExportingClassLoader(ClassLoader parent) {
3739
super(parent);
3840
ModuleSupport.exportAllPackagesTo(getClass(), this);

compiler/src/org.graalvm.compiler.asm.aarch64/src/org/graalvm/compiler/asm/aarch64/AArch64MacroAssembler.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -67,6 +67,7 @@ public class AArch64MacroAssembler extends AArch64Assembler {
6767
// preferred byte alignment for a branch target
6868
public static final int PREFERRED_BRANCH_TARGET_ALIGNMENT = 16;
6969

70+
@SuppressWarnings("this-escape")
7071
public AArch64MacroAssembler(TargetDescription target) {
7172
super(target);
7273
this.neon = new AArch64ASIMDMacroAssembler(this);

compiler/src/org.graalvm.compiler.core.aarch64.test/src/org/graalvm/compiler/core/aarch64/test/AArch64MembarOpTest.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,7 @@ public class AArch64MembarOpTest extends BackendTest {
5555
private final JVMCIBackend providers;
5656
private final CompilationResultBuilder crb;
5757

58+
@SuppressWarnings("this-escape")
5859
public AArch64MembarOpTest() {
5960
this.providers = JVMCI.getRuntime().getHostJVMCIBackend();
6061

compiler/src/org.graalvm.compiler.core.test/src/org/graalvm/compiler/core/test/ConditionalEliminationRegressionTest.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,7 @@ public class ConditionalEliminationRegressionTest extends GraalCompilerTest {
5555

5656
private final SpeculationLog speculationLog;
5757

58+
@SuppressWarnings("this-escape")
5859
public ConditionalEliminationRegressionTest() {
5960
speculationLog = getCodeCache().createSpeculationLog();
6061
}

compiler/src/org.graalvm.compiler.core.test/src/org/graalvm/compiler/core/test/CountedLoopOverflowTest.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,7 @@
5050
public class CountedLoopOverflowTest extends GraalCompilerTest {
5151
private final SpeculationLog speculationLog;
5252

53+
@SuppressWarnings("this-escape")
5354
public CountedLoopOverflowTest() {
5455
speculationLog = getCodeCache().createSpeculationLog();
5556
}

compiler/src/org.graalvm.compiler.core.test/src/org/graalvm/compiler/core/test/GraalCompilerTest.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -373,6 +373,7 @@ public static void resetCache() {
373373
cache.get().clear();
374374
}
375375

376+
@SuppressWarnings("this-escape")
376377
public GraalCompilerTest() {
377378
this.backend = Graal.getRequiredCapability(RuntimeProvider.class).getHostBackend();
378379
this.providers = getBackend().getProviders();

compiler/src/org.graalvm.compiler.core.test/src/org/graalvm/compiler/core/test/InfopointReasonTest.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,7 @@ public class InfopointReasonTest extends GraalCompilerTest {
5151

5252
public static final String[] STRINGS = new String[]{"world", "everyone", "you"};
5353

54+
@SuppressWarnings("this-escape")
5455
public InfopointReasonTest() {
5556
// Call testMethod to ensure all method references are resolved.
5657
testMethod();

compiler/src/org.graalvm.compiler.core/src/org/graalvm/compiler/core/CompilerThread.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@
3131
*/
3232
public class CompilerThread extends Thread {
3333

34+
@SuppressWarnings("this-escape")
3435
public CompilerThread(Runnable r, String namePrefix) {
3536
super(r);
3637
this.setName(namePrefix + "-" + GraalServices.getThreadId(this));

compiler/src/org.graalvm.compiler.core/src/org/graalvm/compiler/core/gen/NodeLIRBuilder.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -145,6 +145,7 @@ public abstract class NodeLIRBuilder implements NodeLIRBuilderTool, LIRGeneratio
145145
private EconomicMap<Class<? extends Node>, List<MatchStatement>> matchRules;
146146
private EconomicMap<Node, Integer> sharedMatchCounts;
147147

148+
@SuppressWarnings("this-escape")
148149
public NodeLIRBuilder(StructuredGraph graph, LIRGeneratorTool gen, NodeMatchRules nodeMatchRules) {
149150
this.gen = (LIRGenerator) gen;
150151
this.nodeMatchRules = nodeMatchRules;

compiler/src/org.graalvm.compiler.core/src/org/graalvm/compiler/core/phases/EconomyHighTier.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@
3030

3131
public class EconomyHighTier extends BaseTier<HighTierContext> {
3232

33+
@SuppressWarnings("this-escape")
3334
public EconomyHighTier() {
3435
CanonicalizerPhase canonicalizer = CanonicalizerPhase.create();
3536
appendPhase(canonicalizer);

0 commit comments

Comments
 (0)