8144223: Move j.l.invoke.{ForceInline, DontInline, Stable} to jdk.internal.vm.annotation package

[Paul Sandoz]

> On 2 Dec 2015, at 21:58, Mandy Chung <mandy.chung at oracle.com> wrote:
> 
> 
>> On Nov 30, 2015, at 9:40 AM, Paul Sandoz <paul.sandoz at oracle.com> wrote:
>> 
>> Please review:
>> 
>> http://cr.openjdk.java.net/~psandoz/jdk9/JDK-8144223-move-stable-force-dont-inline-jdk/webrev/
>> http://cr.openjdk.java.net/~psandoz/jdk9/JDK-8144223-move-stable-force-dont-inline-hotspot/webrev/
> 
> 
> http://cr.openjdk.java.net/~psandoz/jdk9/JDK-8144223-move-stable-force-dont-inline-hotspot/webrev/src/jdk.vm.ci/share/classes/jdk.vm.ci.hotspot/src/jdk/vm/ci/hotspot/Stable.java.cdiff.html
> 
>  32  * This annotation functions as an alias for the jdk.internal.Stable annotation within JVMCI
> 
> s/jdk.internal.Stable/jdk.internal.vm.annotation.Stable
> 
> http://cr.openjdk.java.net/~psandoz/jdk9/JDK-8144223-move-stable-force-dont-inline-jdk/webrev/src/java.base/share/classes/java/lang/invoke/InvokerBytecodeGenerator.java.frames.html
> 
> 1327         mv.visitAnnotation("Ljdk/internal/DontInline;", true);
> 
> need fixing.
> 

Oops that’s embarrassing, i fat fingered the search/replace. Our tests don’t catch such cases of non-existent annotations.

Updated, thanks,
Paul.


> Otherwise, looks good.
> 
> Mandy
>