webrev: use assumeTrue to check for simulator

Doug Simon doug.simon at oracle.com
Mon Jun 30 15:55:59 UTC 2014


Looks ok to me - I’ll integrate it.

On Jun 30, 2014, at 5:10 PM, Eric Caspole <eric.caspole at amd.com> wrote:

> Anyone have time to review this? This will help our internal Sumatra testing run through clean.
> Thanks,
> Eric
> 
> 
> On 06/24/2014 10:09 AM, Caspole, Eric wrote:
>> Hi everybody,
>> I put a new webrev here:
>> 
>> http://cr.openjdk.java.net/~ecaspole/reduce_no_sim/webrev/
>> 
>> The reduce tests already checked in only work when built using the Sumatra JDK and -Dcom.amd.sumatra.offload.immediate=true is used. The HSAIL Simulator does not support HSA group/local memory at this time, so if you run the tests with the JDK and the flag under the simulator, they will fail. Here I added "assumeTrue" to skip the test if running on the simulator which makes it easier for me to work with and test the Sumatra JDK.
>> This change will have no effect when you build Graal using plain JDK 8 since it will not offload the reduce operations.
>> Regards,
>> Eric
>> 



More information about the graal-dev mailing list