scoped options usage in hsail

Doug Simon doug.simon at oracle.com
Mon Nov 11 14:44:23 PST 2013


I’ve integrated this patch.

On Nov 11, 2013, at 11:08 PM, Deneau, Tom <tom.deneau at amd.com> wrote:

> I have posted a small webrev for how we use the scoped options feature in hsail
> 
> 
> 
> http://cr.openjdk.java.net/~tdeneau/graal-webrevs/webrev-scoped-options/webrev/
> 
> 
> 
> The previous way of using scoped options only forced new options on a few tests.  While this was enough to make the existing java 7 junit tests pass, in reality we want to force the options on all tests (this is how we were running things here).
> 
> 
> 
> So I pushed the scoped options overrides down into GraalKernelTester.java which all tests go thru.
> 
> 
> 
> The assumeTrue(aggressiveInliingEnabled) assertions then had to move one layer down, to the runTest routine.
> 
> 
> 
> -- Tom
> 
> 
> 



More information about the graal-dev mailing list