cgen: support -d trace_cgen_fn_decl and -d trace_cgen_gen_fn_decl too

spytheman · spytheman · commit bc4a240b381a · 2025-11-22T13:42:32.000+02:00
diff --git a/vlib/v/gen/c/cgen.v b/vlib/v/gen/c/cgen.v
@@ -284,6 +284,9 @@ mut:
 	curr_comptime_node      &ast.Expr = unsafe { nil } // current `$if` expr
 	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)
 }
 
 @[heap]
@@ -491,7 +494,9 @@ pub fn gen(files []&ast.File, mut table ast.Table, pref_ &pref.Preferences) GenO
 		}
 	} else {
 		util.timing_start('cgen serial processing')
-		for file in files {
+		for fid, file in files {
+			global_g.tid = 0
+			global_g.fid = fid
 			global_g.file = file
 			global_g.gen_file()
 			global_g.cleanups[file.mod.name].drain_builder(mut global_g.cleanup, 100)
@@ -815,6 +820,8 @@ 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
 		file:                  file
 		out:                   strings.new_builder(512000)
 		cheaders:              strings.new_builder(15000)
@@ -925,7 +932,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}')
+		eprintln('> ${@FILE}:${@LINE} | g.file.path: ${g.file.path} | g.tid: ${g.tid:3} | 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
@@ -50,6 +50,9 @@ 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}')
+	}
 	if node.should_be_skipped {
 		return
 	}
@@ -167,6 +170,9 @@ 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}')
+	}
 	// TODO: For some reason, build fails with autofree with this line
 	// as it's only informative, comment it for now
 	// g.gen_attrs(it.attrs)

Original file line number	Diff line number	Diff line change
`@@ -50,6 +50,9 @@ fn (mut g Gen) is_used_by_main(node ast.FnDecl) bool {`
`50`	`50`	`}`
`51`	`51`
`52`	`52`	`fn (mut g Gen) fn_decl(node ast.FnDecl) {`
	`53`	`+ $if trace_cgen_fn_decl ? {`
	`54`	`+ eprintln('> g.tid: ${g.tid:3} \| g.fid: ${g.fid:3} \| g.file.path: ${g.file.path} \| fn_decl: ${node.name}')`
	`55`	`+ }`
`53`	`56`	`if node.should_be_skipped {`
`54`	`57`	`return`
`55`	`58`	`}`
`@@ -167,6 +170,9 @@ fn (mut g Gen) fn_decl(node ast.FnDecl) {`
`167`	`170`	`}`
`168`	`171`
`169`	`172`	`fn (mut g Gen) gen_fn_decl(node &ast.FnDecl, skip bool) {`
	`173`	`+ $if trace_cgen_gen_fn_decl ? {`
	`174`	`+ eprintln('> g.tid: ${g.tid:3} \| g.fid: ${g.fid:3} \| g.file.path: ${g.file.path} \| gen_fn_decl: ${node.name} \| skip: ${skip}')`
	`175`	`+ }`
`170`	`176`	`// TODO: For some reason, build fails with autofree with this line`
`171`	`177`	`// as it's only informative, comment it for now`
`172`	`178`	`// g.gen_attrs(it.attrs)`