Use correct field type when calling GET_INTEGER_VALUE

[dchopra001]

Signed-off-by: Dhruv Chopra Dhruv.C.Chopra@ibm.com

[dchopra001]

For some context, a problem occurs when we try to set a custom port for the JITServer to listen on (using the -XX:JITServerPort=<x>). The code to set this option is here:

openj9/runtime/compiler/control/J9Options.cpp

Lines 1226 to 1232 in 62c6b5b

	if (xxJITServerPortArgIndex >= 0)
	{
	uint32_t port=0;
	IDATA ret = GET_INTEGER_VALUE(xxJITServerPortArgIndex, xxJITServerPortOption, port);
	if (ret == OPTION_OK)
	compInfo->getPersistentInfo()->setJITServerPort(port);
	}

GET_INTEGER_VALUE treats port as a UDATA type (which is an 8 byte field). Since the code above treats it as a uint32_t, this exposes a problem in Z (big endian) where we will always read port as 0. Setting the type of port to UDATA resolves this issue.

While we are in the area, we also change other instances where there is a type mismatch when calling GET_INTEGER_VALUE.

[mpirvu]

LGTM

[mpirvu]

jenkins test sanity zlinuxjit jdk17