# HG changeset patch # User Doug Simon # Date 1404144204 -7200 # Node ID fb90ad461cdbb04801002c0553450493211c7271 # Parent b741f44cb277f251fd8909cf55b79d23fe4c7149 HSAIL: skip certain tests if running on the HSAIL simulator Contributed-by: Eric Caspole diff -r b741f44cb277 -r fb90ad461cdb graal/com.oracle.graal.compiler.hsail.test/src/com/oracle/graal/compiler/hsail/test/lambda/ReduceMaxTest.java --- a/graal/com.oracle.graal.compiler.hsail.test/src/com/oracle/graal/compiler/hsail/test/lambda/ReduceMaxTest.java Mon Jun 30 17:47:45 2014 +0200 +++ b/graal/com.oracle.graal.compiler.hsail.test/src/com/oracle/graal/compiler/hsail/test/lambda/ReduceMaxTest.java Mon Jun 30 18:03:24 2014 +0200 @@ -22,9 +22,11 @@ */ package com.oracle.graal.compiler.hsail.test.lambda; +import com.amd.okra.OkraContext; import static com.oracle.graal.hotspot.HotSpotGraalRuntime.runtime; import com.oracle.graal.hotspot.HotSpotVMConfig; import static org.junit.Assert.*; +import static org.junit.Assume.assumeTrue; import org.junit.*; import java.util.*; @@ -72,6 +74,10 @@ @Test public void testReduce() { + + // The simulator does not support HSA local memory as of June 2014 + assumeTrue(OkraContext.isSimulator() == false); + // Handmade reduce does not support +UseCompressedOops HotSpotVMConfig config = runtime().getConfig(); if (config.useCompressedOops == true || config.useHSAILDeoptimization == true) { diff -r b741f44cb277 -r fb90ad461cdb graal/com.oracle.graal.compiler.hsail.test/src/com/oracle/graal/compiler/hsail/test/lambda/ReduceMinTest.java --- a/graal/com.oracle.graal.compiler.hsail.test/src/com/oracle/graal/compiler/hsail/test/lambda/ReduceMinTest.java Mon Jun 30 17:47:45 2014 +0200 +++ b/graal/com.oracle.graal.compiler.hsail.test/src/com/oracle/graal/compiler/hsail/test/lambda/ReduceMinTest.java Mon Jun 30 18:03:24 2014 +0200 @@ -22,9 +22,11 @@ */ package com.oracle.graal.compiler.hsail.test.lambda; +import com.amd.okra.OkraContext; import static com.oracle.graal.hotspot.HotSpotGraalRuntime.runtime; import com.oracle.graal.hotspot.HotSpotVMConfig; import static org.junit.Assert.*; +import static org.junit.Assume.assumeTrue; import org.junit.*; import java.util.*; @@ -72,6 +74,10 @@ @Test public void testReduce() { + + // The simulator does not support HSA local memory as of June 2014 + assumeTrue(OkraContext.isSimulator() == false); + // Handmade reduce does not support +UseCompressedOops HotSpotVMConfig config = runtime().getConfig(); if (config.useCompressedOops == true || config.useHSAILDeoptimization == true) { diff -r b741f44cb277 -r fb90ad461cdb graal/com.oracle.graal.compiler.hsail.test/src/com/oracle/graal/compiler/hsail/test/lambda/ReduceSumTest.java --- a/graal/com.oracle.graal.compiler.hsail.test/src/com/oracle/graal/compiler/hsail/test/lambda/ReduceSumTest.java Mon Jun 30 17:47:45 2014 +0200 +++ b/graal/com.oracle.graal.compiler.hsail.test/src/com/oracle/graal/compiler/hsail/test/lambda/ReduceSumTest.java Mon Jun 30 18:03:24 2014 +0200 @@ -22,9 +22,11 @@ */ package com.oracle.graal.compiler.hsail.test.lambda; -//import com.oracle.graal.compiler.common.GraalInternalError; +import com.amd.okra.OkraContext; import static com.oracle.graal.hotspot.HotSpotGraalRuntime.runtime; import com.oracle.graal.hotspot.HotSpotVMConfig; +import static org.junit.Assert.assertTrue; +import static org.junit.Assume.assumeTrue; import org.junit.*; import java.util.*; @@ -73,6 +75,10 @@ @Test public void testReduce() { + + // The simulator does not support HSA local memory as of June 2014 + assumeTrue(OkraContext.isSimulator() == false); + // Handmade reduce does not support +UseCompressedOops HotSpotVMConfig config = runtime().getConfig(); if (config.useCompressedOops == true || config.useHSAILDeoptimization == true) { @@ -89,10 +95,10 @@ // Get OptionalInt version kernel sumOffload = evaluate(true); - assert sumStream == sumOffload : "Offload sum is wrong, stream:" + sumStream + " != offload:" + sumOffload; + assertTrue(sumStream == sumOffload); // Get identity version kernel sumOffload = evaluateWithIdentity(true); - assert sumStream == sumOffload : "Offload sum is wrong, stream:" + sumStream + " != offload:" + sumOffload; + assertTrue(sumStream == sumOffload); } }