Merge branch 'master' into gpl-gpu-fft

Bring the PR branch up to date with master to clear the merge conflict. Three gpl files conflicted, all adjacent additions with no logical overlap:

- src/gpl/BUILD: keep both new srcs entries (clockBase + densityGradient)
- src/gpl/src/nesterovBase.cpp: keep both new includes
- src/gpl/src/nesterovPlace.cpp: keep both post-loop calls -- master's
  removeVirtualCts and the GPU host net-box mirror (mirror after the CTS
  removal so any host consumer sees the final state).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Signed-off-by: Minjae Kim <develop.minjae@gmail.com>

Author: dev-minjae

.bazelrc

@@ -1,10 +1,8 @@
 common --enable_bzlmod
-# sv-lang 10.0.bcr.2 (slang at f04e8156 with the foreign_cc cmake -xc++
-# fix) is pending upstream as a PR against bazelbuild/bazel-central-
-# registry: https://github.com/bazelbuild/bazel-central-registry/pull/8987
-# Remove this registry line once the version lands on BCR.
-common --registry=https://raw.githubusercontent.com/oharboe/bazel-central-registry/1beaf797951046d93a2f656d4238c954a21833e5/
-common --registry=https://bcr.bazel.build/
+
+# abc is driven programmatically by OpenROAD, so skip the interactive
+# readline dependency exposed by abc@0.64-yosyshq.bcr.2.
+common --@abc//:use_readline=false
 build --workspace_status_command=tools/workspace_status.sh
 
 # Release builds embed real git version info; dev builds use cache-safe placeholders.
@@ -46,6 +44,16 @@ build  --cxxopt=-fexceptions    --host_cxxopt=-fexceptions
 build --copt "-Wall"               --host_copt "-Wall"
 build --copt "-Wextra"             --host_copt "-Wextra"
 
+# Treat specific warnings as errors
+build --copt "-Werror=array-bounds"               --host_copt "-Werror=array-bounds"
+build --copt "-Werror=dangling"                   --host_copt "-Werror=dangling"
+build --copt "-Werror=dangling-assignment"        --host_copt "-Werror=dangling-assignment"
+build --copt "-Werror=dangling-assignment-gsl"    --host_copt "-Werror=dangling-assignment-gsl"
+build --copt "-Werror=dangling-field"             --host_copt "-Werror=dangling-field"
+build --copt "-Werror=dangling-gsl"               --host_copt "-Werror=dangling-gsl"
+build --copt "-Werror=fortify-source"             --host_copt "-Werror=fortify-source"
+build --copt "-Werror=return-stack-address"       --host_copt "-Werror=return-stack-address"
+
 # ... and disable the warnings we're not interested in.
 build --copt "-Wno-sign-compare"          --host_copt "-Wno-sign-compare"
 build --copt "-Wno-unused-parameter"      --host_copt "-Wno-unused-parameter"
@@ -69,7 +77,6 @@ build --copt "-Wno-cast-function-type-mismatch"  --host_copt "-Wno-cast-function
 build --copt "-Wno-unused-but-set-variable"  --host_copt "-Wno-unused-but-set-variable"
 build --cxxopt "-Wno-deprecated-copy"  --host_cxxopt "-Wno-deprecated-copy"
 build --cxxopt "-Wno-deprecated-copy-with-user-provided-copy"  --host_cxxopt "-Wno-deprecated-copy-with-user-provided-copy"
-build --cxxopt "-Wno-dangling"  --host_cxxopt "-Wno-dangling"
 
 # For 3rd party code: Disable warnings entirely.
 # They are not actionable and just create noise.

.readthedocs.yaml

@@ -15,7 +15,7 @@ build:
     - libasound2
     - graphviz
   tools:
-    python: "3.7"
+    python: "3.10"
     nodejs: "16"
   jobs:
     post_install:

BUILD.bazel

@@ -143,6 +143,7 @@ OPENROAD_COPTS = [
 
 OPENROAD_DEFINES = [
     "BUILD_TYPE=\\\"$(COMPILATION_MODE)\\\"",
+    "BAZEL_BUILD",
     "GPU=false",
     "BUILD_PYTHON=false",
     "ABC_NAMESPACE=abc",
@@ -178,7 +179,6 @@ cc_library(
 # headers to consumers, so direct binary deps are what each Main.cc
 # include needs to see.
 OPENROAD_MAIN_DEPS = [
-    ":openroad_version",
     ":ord",
     ":tcl_readline_setup",
     "//bazel:tcl_library_init",
@@ -188,7 +188,6 @@ OPENROAD_MAIN_DEPS = [
     "//src/web",
     "@abseil-cpp//absl/base:no_destructor",
     "@boost.stacktrace",
-    "@rules_cc//cc/runfiles",  # sets BAZEL_CURRENT_REPOSITORY
     "@tcl_lang//:tcl",
 ]
 
@@ -549,10 +548,14 @@ build_test(
 sh_test(
     name = "lint_tcl_test",
     srcs = ["//bazel:tcl_lint_test.sh"],
-    args = ["$(rootpath //bazel:tclint)"],
+    args = [
+        "$(rootpath //bazel:tclint)",
+        "$(rootpath @git)",
+    ],
     data = [
         "tclint.toml",
         "//bazel:tclint",
+        "@git",
     ],
     tags = ["local"],
 )
@@ -563,10 +566,14 @@ sh_test(
 sh_test(
     name = "fmt_tcl_test",
     srcs = ["//bazel:tcl_fmt_test.sh"],
-    args = ["$(rootpath //bazel:tclfmt)"],
+    args = [
+        "$(rootpath //bazel:tclfmt)",
+        "$(rootpath @git)",
+    ],
     data = [
         "tclint.toml",
         "//bazel:tclfmt",
+        "@git",
     ],
     tags = ["local"],
 )
@@ -590,10 +597,14 @@ sh_binary(
 sh_test(
     name = "lint_bzl_test",
     srcs = ["//bazel:bzl_lint_test.sh"],
-    args = ["$(rootpath @buildifier_prebuilt//:buildifier)"],
+    args = [
+        "$(rootpath @buildifier_prebuilt//:buildifier)",
+        "$(rootpath @git)",
+    ],
     data = [
         "MODULE.bazel",
         "@buildifier_prebuilt//:buildifier",
+        "@git",
     ],
     tags = ["local"],
 )
@@ -604,10 +615,14 @@ sh_test(
 sh_test(
     name = "fmt_bzl_test",
     srcs = ["//bazel:bzl_fmt_test.sh"],
-    args = ["$(rootpath @buildifier_prebuilt//:buildifier)"],
+    args = [
+        "$(rootpath @buildifier_prebuilt//:buildifier)",
+        "$(rootpath @git)",
+    ],
     data = [
         "MODULE.bazel",
         "@buildifier_prebuilt//:buildifier",
+        "@git",
     ],
     tags = ["local"],
 )

CMakeLists.txt

@@ -81,6 +81,8 @@ set(CMAKE_CXX_EXTENSIONS OFF)
 
 list(APPEND CMAKE_MODULE_PATH "${CMAKE_CURRENT_SOURCE_DIR}/cmake")
 
+set_property(DIRECTORY PROPERTY TEST_DISCOVERY_TIMEOUT 60)
+
 # Get version string in OPENROAD_VERSION
 if(NOT OPENROAD_VERSION)
   include(GetGitRevisionDescription)

MODULE.bazel

@@ -7,10 +7,23 @@ module(
 
 # --- Regular dependencies (propagated to downstream consumers via MVS) ---
 
-bazel_dep(name = "abc", version = "0.64-yosyshq.bcr.1")
+bazel_dep(name = "abc", version = "0.64-yosyshq.bcr.2")
+
+# Patch abc's non-readline prompt path to flush stdout and exit on EOF.
+# Without this, yosys's coprocess driver (which waits for the "abc NN> "
+# prompt over a pipe) deadlocks when abc is built with use_readline=false.
+# Drop this override once the fix is in a published BCR abc release.
+single_version_override(
+    module_name = "abc",
+    patch_strip = 1,
+    patches = [
+        "//bazel/abc-patches:0001-mainUtils-flush-prompt-and-handle-eof-without-readline.patch",
+    ],
+)
+
 bazel_dep(name = "abseil-cpp", version = "20260107.1")
 bazel_dep(name = "bazel_skylib", version = "1.7.1")
-bazel_dep(name = "coin-or-lemon", version = "1.3.1")
+bazel_dep(name = "coin-or-lemon", version = "1.3.1.bcr.1")
 bazel_dep(name = "platforms", version = "1.0.0")
 bazel_dep(name = "rules_bison", version = "0.3.1")
 bazel_dep(name = "rules_cc", version = "0.2.18")
@@ -63,11 +76,12 @@ bazel_dep(name = "boost.utility", version = BOOST_VERSION)
 bazel_dep(name = "cudd", version = "3.0.0.bcr.2")
 bazel_dep(name = "eigen", version = "3.4.0.bcr.3")
 bazel_dep(name = "fmt", version = "11.2.0.bcr.1")
+bazel_dep(name = "git", version = "2.54.0")
 bazel_dep(name = "googletest", version = "1.17.0.bcr.2")
 bazel_dep(name = "openmp", version = "21.1.5.bcr.1")
 bazel_dep(name = "or-tools", version = "9.15")
 bazel_dep(name = "spdlog", version = "1.15.1")
-bazel_dep(name = "sv-lang", version = "10.0.bcr.2")
+bazel_dep(name = "sv-lang", version = "10.0.1-20260316-f04e8156")
 bazel_dep(name = "tcl_lang", version = "9.0.2.bcr.1")
 bazel_dep(name = "tcmalloc", version = "0.0.0-20250927-12f2552")
 bazel_dep(name = "yaml-cpp", version = "0.9.0")
@@ -144,18 +158,7 @@ archive_override(
 )
 
 bazel_dep(name = "yosys", version = "0.62.bcr.2", dev_dependency = True)
-
-# yosys-slang is not on BCR. Pin to a commit on povik/yosys-slang master
-# that has the upstream Bazel build (povik/yosys-slang#310) and the
-# slang.so visibility fix (povik/yosys-slang#311). Submodules pull in
-# vendored slang and fmt sources.
-bazel_dep(name = "yosys-slang", dev_dependency = True)
-git_override(
-    module_name = "yosys-slang",
-    commit = "7753ea70431d85929292b90c33b32f6dbdb3b048",
-    init_submodules = True,
-    remote = "https://github.com/povik/yosys-slang.git",
-)
+bazel_dep(name = "yosys-slang", version = "0.0.0", dev_dependency = True)
 
 # --- Extensions ---
 
@@ -252,3 +255,12 @@ single_version_override(
     patches = ["//bazel:boost_context_disable_parse_headers.patch"],
     version = BOOST_VERSION,
 )
+
+# scip 9.2.3's tinycthread uses bare C11 <threads.h> names. Backport
+# scipopt/scip@cc917e07 (TNY_ prefix) until a fixed release lands in BCR.
+single_version_override(
+    module_name = "scip",
+    patch_strip = 1,
+    patches = ["//bazel:scip_tinycthread_tny_prefix.patch"],
+    version = "9.2.3",
+)