]> git.djapps.eu Git - pkg/ggml/sources/whisper.cpp/commitdiff
CANN: Define `cann_graph_update_required` before macro (llama/17434)
authorRaul Torres <redacted>
Mon, 24 Nov 2025 02:02:52 +0000 (02:02 +0000)
committerGeorgi Gerganov <redacted>
Fri, 12 Dec 2025 15:53:06 +0000 (17:53 +0200)
**Description of the problem**

`cann_graph_update_required` is redundantly defined and
initialized as `false` inside two mutually exclusive macro branches.

**Proposed solution**

Define it right before the macro so that it could serve both
branches.

ggml/src/ggml-cann/ggml-cann.cpp

index 5cbf5683e1dcdc16c2a803b32c1433b042689c1e..3c67c48ffa11f3fc425d9d88aa54105b97ce4c2d 100644 (file)
@@ -2303,9 +2303,9 @@ static enum ggml_status ggml_backend_cann_graph_compute(ggml_backend_t backend,
     // calculate rope cache for fist layer in current device.
     cann_ctx->rope_cache.cached = false;
 
+    bool cann_graph_update_required = false;
 #ifdef USE_ACL_GRAPH
     bool use_cann_graph             = true;
-    bool cann_graph_update_required = false;
 
     static bool prefill_use_graph = parse_bool(get_env("GGML_CANN_PREFILL_USE_GRAPH").value_or(""));
     if (!prefill_use_graph) {
@@ -2336,7 +2336,6 @@ static enum ggml_status ggml_backend_cann_graph_compute(ggml_backend_t backend,
     }
 #else
     bool use_cann_graph             = false;
-    bool cann_graph_update_required = false;
 #endif  // USE_ACL_GRAPH
     evaluate_and_capture_cann_graph(cann_ctx, cgraph, use_cann_graph, cann_graph_update_required);