Include inference time in --trace-compile-timing (#59220)

topolarity · KristofferC · commit 4aca0d6d0d53 · 2025-09-02T08:15:09.000+02:00
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)
diff --git a/src/gf.c b/src/gf.c
@@ -3519,9 +3519,8 @@ jl_code_instance_t *jl_compile_method_internal(jl_method_instance_t *mi, size_t
         }
 
         JL_GC_PUSH1(&codeinst);
-        double compile_time = jl_hrtime();
         int did_compile = jl_compile_codeinst(codeinst);
-        compile_time = jl_hrtime() - compile_time;
+        double compile_time = jl_hrtime() - start;
 
         if (jl_atomic_load_relaxed(&codeinst->invoke) == NULL) {
             // Something went wrong. Bail to the fallback path.

Original file line number	Diff line number	Diff line change
`@@ -3519,9 +3519,8 @@ jl_code_instance_t jl_compile_method_internal(jl_method_instance_t mi, size_t`
`3519`	`3519`	`}`
`3520`	`3520`
`3521`	`3521`	`JL_GC_PUSH1(&codeinst);`
`3522`		`- double compile_time = jl_hrtime();`
`3523`	`3522`	`int did_compile = jl_compile_codeinst(codeinst);`
`3524`		`- compile_time = jl_hrtime() - compile_time;`
	`3523`	`+ double compile_time = jl_hrtime() - start;`
`3525`	`3524`
`3526`	`3525`	`if (jl_atomic_load_relaxed(&codeinst->invoke) == NULL) {`
`3527`	`3526`	`// Something went wrong. Bail to the fallback path.`