Remove withfield/aconst_init uses from ValueTypeTests

- Remove ACONST_INIT and WITHFIELD bytecode uses from tests
- Consolidate value and reference class generator creation methods now that both use <init> and putfield
- Remove wither tests. There is no equivalent behavior in the new spec.
- Remove testWithFieldOnNonValueType, testWithFieldOnNull, testWithFieldOnNonExistentClass, testUnresolvedWithFieldUse

Signed-off-by: Theresa Mammarella <[email protected]>

Author: theresa-m

test/functional/Valhalla/src/org/openj9/test/lworld/ValhallaUtils.java

@@ -35,10 +35,6 @@ public class ValhallaUtils {
 	/* workaround till the new ASM is released */
 	static final int ACC_IDENTITY = 0x20;
 
-	/* these can be removed with qtypes tests */
-	static final int ACONST_INIT = 203;
-	static final int WITHFIELD = 204;
-
 	/* ImplicitCreation flags */
 	static final int ACC_DEFAULT = 0x1;
 	static final int ACC_NON_ATOMIC = 0x2;

test/functional/Valhalla/src_qtypes/org/openj9/test/lworld/DDRValueTypeTest.java

@@ -47,7 +47,7 @@ private static void createAndCheckValueType() throws Throwable {
 		Class assortedValueWithSingleAlignmentClass = ValueTypeGenerator.generateValueClass("AssortedValueWithSingleAlignment", ValueTypeTests.typeWithSingleAlignmentFields);
 
 		MethodHandle makeAssortedValueWithSingleAlignment = MethodHandles.lookup().findStatic(assortedValueWithSingleAlignmentClass,
-			"makeValueGeneric", MethodType.methodType(Object.class, Object.class,
+			"makeObjectGeneric", MethodType.methodType(Object.class, Object.class,
 						Object.class, Object.class, Object.class, Object.class, Object.class));
 
 		Object[] altFields = {