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

Part 3 of adding lw5 ValueTypeTests #18348

Merged
merged 1 commit into from
Oct 30, 2023

Conversation

theresa-m
Copy link
Contributor

The following tests are added:

testCreateLargeValueObjectAndMegaValueObject
testCreateLargeRefObjectAndMegaRefObject
testAssortedValueWithLongAlignment
testAssortedRefWithLongAlignment
testAssortedValueWithObjectAlignment
testAssortedRefWithObjectAlignment
testAssortedValueWithSingleAlignment
testAssortedRefWithSingleAlignment

testStaticFieldsWithSingleAlignment
testStaticFieldsWithLongAlignment
testStaticFieldsWithObjectAlignment

testGCFlattenedPoint2DArray
testGCFlattenedValueArrayWithSingleAlignment
testGCFlattenedValueArrayWithObjectAlignment
testGCFlattenedValueArrayWithLongAlignment
testGCFlattenedLargeObjectArray
testGCFlattenedMegaObjectArray

The following tests will never be added to lw5 ValueTypeTests:

testInvalidNestedField <- replaced by preload tests in ValhallaAttributeTests testNullWritesOnNonNullableArrays <- this will be added to ValhallaAttributeTests once support exists for nullrestricted arrays testNoneValueQTypeAsNestedField <- replaced by nullrestricted tests in ValhallaAttributeTests

withfield tests since this bytecode is being removed: testWithFieldOnNonValueType

testWithFieldOnNull
testWithFieldOnNonExistentClass

isPrimitiveClass tests since the primitive bytecode is going away: testIsPrimitiveClassOnRef

testIsPrimitiveClassOnValueType
testIsPrimitiveClassOnInterface
testIsPrimitiveClassOnAbstractClass
testIsPrimitiveOnValueArrayClass
testIsPrimitiveOnRefArrayClass

@theresa-m theresa-m added comp:test project:valhalla Used to track Project Valhalla related work labels Oct 25, 2023
@theresa-m theresa-m force-pushed the valuetypetests_3 branch 2 times, most recently from 3633eb1 to 975dfd4 Compare October 25, 2023 20:46
@hangshao0
Copy link
Contributor

@theresa-m There is a line ending check failure.

The following tests are added:

testCreateLargeValueObjectAndMegaValueObject
testCreateLargeRefObjectAndMegaRefObject
testAssortedValueWithLongAlignment
testAssortedRefWithLongAlignment
testAssortedValueWithObjectAlignment
testAssortedRefWithObjectAlignment
testAssortedValueWithSingleAlignment
testAssortedRefWithSingleAlignment

testStaticFieldsWithSingleAlignment
testStaticFieldsWithLongAlignment
testStaticFieldsWithObjectAlignment

testGCFlattenedPoint2DArray
testGCFlattenedValueArrayWithSingleAlignment
testGCFlattenedValueArrayWithObjectAlignment
testGCFlattenedValueArrayWithLongAlignment
testGCFlattenedLargeObjectArray
testGCFlattenedMegaObjectArray

The following tests will never be added to lw5 ValueTypeTests:

testInvalidNestedField <- replaced by preload tests in ValhallaAttributeTests
testNullWritesOnNonNullableArrays <- this will be added to ValhallaAttributeTests once support exists for nullrestricted arrays
testNoneValueQTypeAsNestedField <- replaced by nullrestricted tests in ValhallaAttributeTests

withfield tests since this bytecode is being removed:
testWithFieldOnNonValueType
testWithFieldOnNull
testWithFieldOnNonExistentClass

isPrimitiveClass tests since the primitive bytecode is going away:
testIsPrimitiveClassOnRef
testIsPrimitiveClassOnValueType
testIsPrimitiveClassOnInterface
testIsPrimitiveClassOnAbstractClass
testIsPrimitiveOnValueArrayClass
testIsPrimitiveOnRefArrayClass

Signed-off-by: Theresa Mammarella <Theresa.T.Mammarella@ibm.com>
@theresa-m
Copy link
Contributor Author

Sorry thought I got these all on Friday. The line check is passing now.

@hangshao0
Copy link
Contributor

Jenkins test sanity,extended winval jdknext

@hangshao0
Copy link
Contributor

Jenkins test sanity,extended aixval jdknext

@hangshao0 hangshao0 merged commit a80c01f into eclipse-openj9:master Oct 30, 2023
9 checks passed
@theresa-m theresa-m deleted the valuetypetests_3 branch December 6, 2023 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp:test project:valhalla Used to track Project Valhalla related work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants