Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[GR-53454] Warn when @ReportPolymorphism is used incorrectly #9160

Closed
wants to merge 2 commits into from

Conversation

graalvmbot
Copy link
Collaborator

  • Remove duplicate .required() check.

It looks like this:

truffleruby/src/main/java/org/truffleruby/cext/CExtNodes.java:1054: warning: This node uses @ReportPolymorphism but has a single specialization, so the annotation has no effect.
    public abstract static class RbGvGetNode extends PrimitiveArrayArgumentsNode {
                           ^
truffleruby/src/main/java/org/truffleruby/core/array/ArrayNodes.java:1803: warning: This node uses @ReportPolymorphism but has a single specialization, so the annotation has no effect.
    public abstract static class ReplaceNode extends CoreMethodNode {
                           ^

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Jun 20, 2024
@graalvmbot graalvmbot force-pushed the bd/GR-53454 branch 4 times, most recently from 3b8fe71 to c2e2e8f Compare June 24, 2024 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants