RFR: 8138732: Rename @HotSpotIntrinsicCandidate to @IntrinsicCandidate and move it to the jdk.internal.vm.annotation package
Philippe Marschall
philippe.marschall at gmail.com
Wed Sep 9 09:45:37 UTC 2020
On Mon, Sep 7, 2020 at 7:54 PM Alan Bateman <alanb at openjdk.java.net> wrote:
>
> On Mon, 7 Sep 2020 09:44:09 GMT, Philippe Marschall <github.com+471021+marschall at openjdk.org> wrote:
>
> > Hello, newbie here
> >
> > I picked JDK-8138732 to work on because it has a "starter" label and I believe I understand what to do.
> >
> > - I tried to update the copyright year to 2020 in every file.
> > - I decided to change `@since` from 9 to 16 since it is a new annotation name in a new package.
> > - I tried to keep code changes to a minimum, eg not change to imports if fully qualified class names are used instead of
> > imports. In some cases I did minor reordering of imports to keep them sorted alphabetically.
> > - All tier1 tests pass.
> > - One jpackage/jlink tier2 test fails but I don't believe it is related.
> > - Some tier3 Swing tests fail but I don't think they are related.
>
> This one probably needs discussion on hotspot-dev to get agreement on the rename/move. It might need coordination with
> Graal.
I sent out mails to hotspot-dev and graal-dev.
> If the change does go ahead then please check if java.base's module-info.java still needs to export jdk.internal
> to jdk.jfr, I suspect that qualified export can be removed.
You're correct we can replace the jdk.internal export with the
jdk.internal.vm.annotation export. I updated the PR accordingly.
Cheers
Philippe
More information about the security-dev
mailing list