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

gc-debug: fix time unit in jl_print_gc_stats #50417

Merged
merged 1 commit into from
Jul 5, 2023

Conversation

inkydragon
Copy link
Sponsor Member

jl_hrtime() return nanosec. Then ptime is also in nanosec.
Need to convert unit to seconds with the jl_ns2s function.

julia/src/sys.c

Lines 461 to 462 in 7fc8646

// Returns time in nanosec
JL_DLLEXPORT uint64_t jl_hrtime(void) JL_NOTSAFEPOINT

process_t0 = jl_hrtime();

NOTE: those codes need flag: CFLAGS = -DGC_FINAL_STATS

@vtjnash vtjnash merged commit 23c0418 into JuliaLang:master Jul 5, 2023
4 of 6 checks passed
@inkydragon inkydragon deleted the mem-stat branch July 5, 2023 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants