Skip to content

Fix the output of floating point values in writeGPInfo() - #19463

Merged
pshipton merged 1 commit into
eclipse-openj9:masterfrom
knn-k:fixWriteGPInfo
May 8, 2024
Merged

Fix the output of floating point values in writeGPInfo()#19463
pshipton merged 1 commit into
eclipse-openj9:masterfrom
knn-k:fixWriteGPInfo

Conversation

@knn-k

@knn-k knn-k commented May 8, 2024

Copy link
Copy Markdown
Contributor

This commit changes the output of floating point values in writeGPInfo(). Floating point values are printed as "registerName value", and this commit changes it to "registerName=value" to align with outputs of other kinds.

This commit changes the output of floating point values in writeGPInfo().
Floating point values are printed as "registerName value", and this
commit changes it to "registerName=value" to align with outputs of other
kinds.

Signed-off-by: KONNO Kazuhiro <konno@jp.ibm.com>
@knn-k knn-k added the comp:vm label May 8, 2024
@knn-k
knn-k force-pushed the fixWriteGPInfo branch from 81ccf1d to fa632a3 Compare May 8, 2024 05:37
@knn-k

knn-k commented May 8, 2024

Copy link
Copy Markdown
Contributor Author

Jenkins test sanity xmac jdk11

@knn-k

knn-k commented May 8, 2024

Copy link
Copy Markdown
Contributor Author

Jenkins test sanity alinux64 jdk11

@knn-k

knn-k commented May 8, 2024

Copy link
Copy Markdown
Contributor Author

See #19456 (aarch64) and #19438 (power) for example.

GPRs:
x0=000000013C964500 x1=00000001705C27C8 x2=0000000150133C00 x3=0000000150133C00

FPRs:
v0 00000001340a0309 (f: 873071360.000000, d: 2.553350e-314)
v1 000000013680bfb8 (f: 914407360.000000, d: 2.573773e-314)
GPRs:
R0=00007FFFA29230D4 R1=00007FFF87FCAA30 R2=00007FFFA8946C00 R3=0000000000230F00

FPRs:
FPR0 0000000000000004 (f: 4.000000, d: 1.976263e-323)
FPR1 4055558000000000 (f: 0.000000, d: 8.533594e+01)

@knn-k
knn-k marked this pull request as ready for review May 8, 2024 10:17
@pshipton
pshipton merged commit b3944af into eclipse-openj9:master May 8, 2024
@knn-k
knn-k deleted the fixWriteGPInfo branch May 8, 2024 22:50
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