Small change to build Graal with Lambda repo

[Christian Thalinger]

On Apr 26, 2013, at 7:26 AM, Doug Simon <doug.simon at oracle.com> wrote:

> 
> On Apr 26, 2013, at 3:52 PM, "Caspole, Eric" <Eric.Caspole at amd.com> wrote:
> 
>> Hi everybody,
>> I have been experimenting to build Graal against the jdk 8 lambda tip repo. It seems the args for sun.reflect.Reflection.getCallerClass() changed between JDK 7 and 8.
>> 
>> This one line change allows the build to succeed and I can use this JDK 8 Graal build in NetBeans to run the Junits.
>> http://cr.openjdk.java.net/~ecaspole/reflection_args_for_lambda_jdk/webrev.00/
>> 
>> I am not sure there is a JDK 8 branch for Graal, can this be integrated without breaking JDK 7?
> 
> I don't think so. However, if this is the only thing preventing one using the latest JDK 8, I vote we just delete the test.

Yes, we should remove the test because it's an internal API that can be changed any time.  For that exact same reason I didn't add a test for:

GRAAL-246: intrinsify Reflection.getCallerClass

-- Chris

> The alternative is to use a bunch of reflection to make the right call based on the underlying JDK which kind of defeats the intention of the JTT tests which should only stress a limited number of features.
> 
> -Doug