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

DDR: Fix printing of class static field values #4366

Merged
merged 4 commits into from
Jan 22, 2019

Conversation

keithc-ca
Copy link
Contributor

  • access a minimum of 32-bits (as the VM does) to handle byte ordering properly
  • !j9statics output now suggests use of !j9romstaticfieldshape instead of non-existent commands !j9romstaticdoublefieldshape, !j9romstaticsinglefieldshape or !j9romstaticstringfieldshape
  • remove unnecessary casts
  • fix test for NULL pointer
  • correct C/C++ source references

Signed-off-by: Keith W. Campbell <keithc@ca.ibm.com>
Signed-off-by: Keith W. Campbell <keithc@ca.ibm.com>
Signed-off-by: Keith W. Campbell <keithc@ca.ibm.com>
* access a minimum of 32-bits to handle byte ordering properly
* !j9statics output now suggests use of j9romstaticfieldshape
  instead of non-existent commands j9romstaticdoublefieldshape,
  j9romstaticsinglefieldshape or j9romstaticstringfieldshape

Signed-off-by: Keith W. Campbell <keithc@ca.ibm.com>
@pshipton
Copy link
Member

jenkins test all zlinux jdk11

@pshipton
Copy link
Member

jenkins test sanity zlinux jdk11

1 similar comment
@pshipton
Copy link
Member

jenkins test sanity zlinux jdk11

@pshipton
Copy link
Member

jenkins test extended zlinux jdk11

@pshipton pshipton merged commit b5e3c2a into eclipse-openj9:master Jan 22, 2019
@keithc-ca keithc-ca deleted the endian branch January 22, 2019 14:31
@keithc-ca keithc-ca mentioned this pull request Jan 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants