-
-
Notifications
You must be signed in to change notification settings - Fork 316
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
Exclude 2 vector tests on arm for jdk17 #2992
Comments
Hello @sophia-guo . I'm Harriet,an outreachy Internship applicant. Can I take on this issue? |
Hello @sophia-guo . Above is my proposed solution to the issue adoptium#2992 "Exclude 2 vector tests on arm for jdk17". Kindly review my proposed changes. Thank you
Thanks @HarrietAkot . |
@HarrietAkot Please create a PR so we have review it. Thanks. |
Thanks @sophia-guo . I tried to push changes to the repository but I keep getting a 403 forbidden errorr. Alternatively, I directly edited the file on this commit. 97016c1 The error is below |
You can find a general manual about open source contributions at https://github.com/firstcontributions/first-contributions which should help guide you... you'll need to fork the repo, clone the repo, create a branch, make your changes in your branch, commit the changes then push the changes and create a PR (as per that first-contributions guide). |
Thank you for the detailed tutorial @smlambert. |
Tests jdk/incubator/vector/Byte256VectorLoadStoreTests.java.Byte256VectorLoadStoreTests
jdk/incubator/vector/ByteMaxVectorLoadStoreTests.java.ByteMaxVectorLoadStoreTests failed the job https://ci.adoptopenjdk.net/job/Test_openjdk17_hs_extended.openjdk_arm_linux/ with crash.
Issue related: #2874
To exclude: adding two exclusion statements in https://github.com/adoptium/aqa-tests/blob/master/openjdk/excludes/ProblemList_openjdk17.txt (likely just after Line 328).
The text was updated successfully, but these errors were encountered: