[llvm] [docs] Fix default Sphinx worker count (PR #217429)

Reid Kleckner via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 19 11:41:54 PDT 2026


https://github.com/rnk created https://github.com/llvm/llvm-project/pull/217429

I renamed the variable at the last minute, and the search and replace didn't catch all instances. Follow-up to #217161 / 0e6f9b26868c37cfb27fdc99af0524ed78e1dfd8.

>From 07ce2a8a89bcce10a509980fd7cf7e28c38e36a2 Mon Sep 17 00:00:00 2001
From: Reid Kleckner <rkleckner at nvidia.com>
Date: Wed, 19 Aug 2026 18:33:46 +0000
Subject: [PATCH] [docs] Fix default Sphinx worker count

The rename from LLVM_SPHINX_BUILD_JOBS to LLVM_SPHINX_THREADS left the cache initialization reading the old variable name. That overwrote the computed default with an empty value, so the sphinx-build -j flag was omitted unless LLVM_SPHINX_THREADS was set explicitly.

Use the computed LLVM_SPHINX_THREADS value when creating the cache entry.
---
 llvm/cmake/modules/AddSphinxTarget.cmake | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/llvm/cmake/modules/AddSphinxTarget.cmake b/llvm/cmake/modules/AddSphinxTarget.cmake
index 6b3987cbbc7c0..197a1552d6b4e 100644
--- a/llvm/cmake/modules/AddSphinxTarget.cmake
+++ b/llvm/cmake/modules/AddSphinxTarget.cmake
@@ -163,7 +163,7 @@ function (add_sphinx_target builder project)
     math(EXPR LLVM_SPHINX_THREADS
          "(${number_of_logical_cores} + 1) / 2")
   endif()
-  set(LLVM_SPHINX_THREADS "${LLVM_SPHINX_BUILD_JOBS}"
+  set(LLVM_SPHINX_THREADS "${LLVM_SPHINX_THREADS}"
       CACHE STRING "Define the number of parallel jobs for each Sphinx build.")
   if (LLVM_SPHINX_THREADS)
     set(sphinx_jobs_flag -j ${LLVM_SPHINX_THREADS})



More information about the llvm-commits mailing list