cgen: show the v_gettid() hex with -d trace_cgen_fn_decl and -d trace_cgen_gen_fn_decl

spytheman · spytheman · commit b968bf0e8112 · 2025-11-23T19:33:55.000+02:00
diff --git a/vlib/v/gen/c/cgen.v b/vlib/v/gen/c/cgen.v
@@ -285,8 +285,8 @@ mut:
 	is_builtin_overflow_mod bool
 	do_int_overflow_checks  bool // outside a `@[ignore_overflow] fn abc() {}` or a function in `builtin.overflow`
 	//
-	tid int // the thread id of the file processor in the thread pool (log it to debug issues in parallel cgen)
-	fid int // the index of ast.File that is currently processed (log it to debug issues in parallel cgen)
+	tid string // the thread id of the file processor in the thread pool (log it to debug issues in parallel cgen)
+	fid int    // the index of ast.File that is currently processed (log it to debug issues in parallel cgen)
 }
 
 @[heap]
@@ -317,6 +317,8 @@ pub fn gen(files []&ast.File, mut table ast.Table, pref_ &pref.Preferences) GenO
 	}
 	mut reflection_strings := map[string]int{}
 	mut global_g := Gen{
+		fid:                  -1
+		tid:                  v_gettid().hex()
 		file:                 unsafe { nil }
 		out:                  strings.new_builder(512000)
 		cheaders:             strings.new_builder(15000)
@@ -495,7 +497,6 @@ pub fn gen(files []&ast.File, mut table ast.Table, pref_ &pref.Preferences) GenO
 	} else {
 		util.timing_start('cgen serial processing')
 		for fid, file in files {
-			global_g.tid = 0
 			global_g.fid = fid
 			global_g.file = file
 			global_g.gen_file()
@@ -504,6 +505,7 @@ pub fn gen(files []&ast.File, mut table ast.Table, pref_ &pref.Preferences) GenO
 			global_g.tmp_count = 0
 		}
 		util.timing_measure('cgen serial processing')
+		global_g.fid = -1
 
 		util.timing_start('cgen unification')
 	}
@@ -821,7 +823,7 @@ fn cgen_process_one_file_cb(mut p pool.PoolProcessor, idx int, wid int) &Gen {
 	mut global_g := unsafe { &Gen(p.get_shared_context()) }
 	mut g := &Gen{
 		fid:                   idx
-		tid:                   wid
+		tid:                   v_gettid().hex()
 		file:                  file
 		out:                   strings.new_builder(512000)
 		cheaders:              strings.new_builder(15000)
@@ -932,7 +934,7 @@ pub fn (mut g Gen) free_builders() {
 
 pub fn (mut g Gen) gen_file() {
 	$if trace_cgen ? {
-		eprintln('> ${@FILE}:${@LINE} | g.file.path: ${g.file.path} | g.tid: ${g.tid:3} | g.fid: ${g.fid:3}')
+		eprintln('> ${@FILE}:${@LINE} | g.file.path: ${g.file.path} | g.tid: ${g.tid} | g.fid: ${g.fid:3}')
 	}
 	g.timers.start('cgen_file ${g.file.path}')
 	g.unique_file_path_hash = fnv1a.sum64_string(g.file.path)
diff --git a/vlib/v/gen/c/fn.v b/vlib/v/gen/c/fn.v
@@ -51,7 +51,7 @@ fn (mut g Gen) is_used_by_main(node ast.FnDecl) bool {
 
 fn (mut g Gen) fn_decl(node ast.FnDecl) {
 	$if trace_cgen_fn_decl ? {
-		eprintln('>   g.tid: ${g.tid:3} | g.fid: ${g.fid:3} | g.file.path: ${g.file.path} | fn_decl: ${node.name}')
+		eprintln('>   g.tid: ${g.tid} | g.fid: ${g.fid:3} | g.file.path: ${g.file.path} | fn_decl: ${node.name}')
 	}
 	if node.should_be_skipped {
 		return
@@ -171,7 +171,7 @@ fn (mut g Gen) fn_decl(node ast.FnDecl) {
 
 fn (mut g Gen) gen_fn_decl(node &ast.FnDecl, skip bool) {
 	$if trace_cgen_gen_fn_decl ? {
-		eprintln('>   g.tid: ${g.tid:3} | g.fid: ${g.fid:3} | g.file.path: ${g.file.path} | gen_fn_decl: ${node.name} | skip: ${skip}')
+		eprintln('>   g.tid: ${g.tid} | g.fid: ${g.fid:3} | g.file.path: ${g.file.path} | gen_fn_decl: ${node.name} | skip: ${skip}')
 	}
 	// TODO: For some reason, build fails with autofree with this line
 	// as it's only informative, comment it for now
