fix(moe): unconditional moe_shared_expert label (non-Metal build)

unamedkr · claude · unamedkr · commit f81fb2fcd4bb · 2026-04-26T16:17:05.000+09:00
The label was inside #ifdef TQ_HAS_METAL but referenced via goto from
non-Metal code paths (R5+ MoE fast-path skip). Non-Metal builds failed
with 'use of undeclared label'. Make the label unconditional.

Discovered during R8 clean rebuild bisect: cmake without -DTQ_BUILD_METAL=ON
hit this build error.

Co-Authored-By: Claude Opus 4.7 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/src/engine/tq_moe.c b/src/engine/tq_moe.c
@@ -1245,9 +1245,10 @@ moe_cpu_fallback: ;
         }
     }
 
-#ifdef TQ_HAS_METAL
 moe_shared_expert:
-#endif
+    /* Label is referenced by goto from non-Metal code paths (R5+ MoE
+     * fast-path skip, line ~1065). Keep the label unconditional so
+     * non-Metal builds compile. */
     /* Step 4: Shared expert (always-active, if present).
      * TQ_MOE_NO_SHARED=1 disables it entirely for A/B testing against
      * llama.cpp's presumed identical path. If disabling improves

Original file line number	Diff line number	Diff line change
`@@ -1245,9 +1245,10 @@ moe_cpu_fallback: ;`
`1245`	`1245`	`}`
`1246`	`1246`	`}`
`1247`	`1247`
`1248`		`-#ifdef TQ_HAS_METAL`
`1249`	`1248`	`moe_shared_expert:`
`1250`		`-#endif`
	`1249`	`+ /* Label is referenced by goto from non-Metal code paths (R5+ MoE`
	`1250`	`+ * fast-path skip, line ~1065). Keep the label unconditional so`
	`1251`	`+ * non-Metal builds compile. */`
`1251`	`1252`	`/* Step 4: Shared expert (always-active, if present).`
`1252`	`1253`	`* TQ_MOE_NO_SHARED=1 disables it entirely for A/B testing against`
`1253`	`1254`	`* llama.cpp's presumed identical path. If disabling improves`