Skip to content

Commit 16adcf1

Browse files
topolarityKristofferC
authored andcommitted
Include inference time in --trace-compile-timing (#59220)
It's important for this measurement to include indirect compilation (not just "self" time), since the `--trace-compile` log only reports compilation triggers / entrypoints (not all compiled code). (cherry picked from commit 888c2c3)
1 parent 5f767fc commit 16adcf1

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

src/gf.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3519,9 +3519,8 @@ jl_code_instance_t *jl_compile_method_internal(jl_method_instance_t *mi, size_t
35193519
}
35203520

35213521
JL_GC_PUSH1(&codeinst);
3522-
double compile_time = jl_hrtime();
35233522
int did_compile = jl_compile_codeinst(codeinst);
3524-
compile_time = jl_hrtime() - compile_time;
3523+
double compile_time = jl_hrtime() - start;
35253524

35263525
if (jl_atomic_load_relaxed(&codeinst->invoke) == NULL) {
35273526
// Something went wrong. Bail to the fallback path.

0 commit comments

Comments
 (0)