Merge remote-tracking branch 'origin/master' into drt_abutment_debug

Author: bnmfw

.bazelignore

@@ -1 +1,2 @@
 build/
+debug/

.bazelrc

@@ -1,10 +1,19 @@
 common --enable_bzlmod
+common --enable_workspace  # explicitly currently still needing this
 
 # suppress distracting warning about newer module
 # versions found in dependency graph.
 common --check_direct_dependencies=off
 
+# Test timeouts for various levels.
+test --test_timeout=300,1800,1800,9600
+
+# bazel 7 is somewhat forgiving for glob patterns that don't
+# match, but bazel 8 will be strict. So start now.
+common --incompatible_disallow_empty_glob
+
 build --cxxopt "-std=c++17"        --host_cxxopt "-std=c++17"
+build --cxxopt "-xc++"             --host_cxxopt "-xc++"
 build --cxxopt "-ffp-contract=off" --host_cxxopt "-ffp-contract=off"
 
 # allow exceptions.

.github/workflows/github-actions-bazel-build.yml

@@ -32,9 +32,14 @@ jobs:
 
       - name: Build
         run: |
-          # For now, just run, but don't fail the whole CI, as bazel build is
-          # experimental at this stage.
-          bazel test -c opt --keep_going --show_timestamps --test_output=errors --curses=no --force_pic ... || /bin/true
+          bazel test \
+           -c opt \
+           --keep_going \
+           --show_timestamps \
+           --test_output=errors \
+           --curses=no \
+           --force_pic \
+           ...
 
       - name: Smoke test
         run: |

AUTHORS

@@ -17,3 +17,4 @@ Precision Innovations, Inc.
 The Regents of the University of California
 The Regents of the University of Michigan
 The Regents of the University of Minnesota
+Øyvind Harboe - Zylin AS, Ascenium AS

BUILD.bazel

@@ -64,6 +64,7 @@ cc_binary(
         ":openroad_lib_private",
         "@rules_cc//cc/runfiles",
     ],
+    malloc = "@tcmalloc//tcmalloc",
 )
 
 cc_library(
@@ -1201,6 +1202,7 @@ cc_binary(
         ":opensta_lib",
         "@tk_tcl//:tcl",
     ],
+    malloc = "@tcmalloc//tcmalloc",
 )
 
 cc_library(

CMakeLists.txt

@@ -138,10 +138,20 @@ endif()
 add_subdirectory(src)
 add_subdirectory(test)
 
-# After all compiled, look for the respective flags
+# Not currently used but may become useful again with Kokkos
 target_compile_definitions(openroad PRIVATE GPU)
-target_compile_definitions(openroad PRIVATE BUILD_PYTHON)
-target_compile_definitions(openroad PRIVATE BUILD_GUI)
+
+if(BUILD_PYTHON)
+  target_compile_definitions(openroad PRIVATE BUILD_PYTHON=1)
+else()
+  target_compile_definitions(openroad PRIVATE BUILD_PYTHON=0)
+endif()
+
+if(BUILD_GUI)
+  target_compile_definitions(openroad PRIVATE BUILD_GUI=1)
+else()
+  target_compile_definitions(openroad PRIVATE BUILD_GUI=0)
+endif()
 
 ####################################################################
 

MODULE.bazel

@@ -9,7 +9,17 @@ bazel_dep(name = "platforms", version = "0.0.11")
 bazel_dep(name = "bazel_skylib", version = "1.7.1")
 bazel_dep(name = "rules_flex", version = "0.3.1")
 bazel_dep(name = "rules_bison", version = "0.3.1")
+
+# The current rules_cc 0.1.1 hardcodes script locations to `#!/bin/bash`
+# instead of using `#!/usr/bin/env bash`.
+# This is fixed by https://github.com/bazelbuild/rules_cc/pull/306 but
+# is not released yet. Use rules_cc from that commit.
 bazel_dep(name = "rules_cc", version = "0.1.1")
+git_override(
+    module_name = "rules_cc",
+    commit = "d74915024017250e46d95e91a3defc34174effe0",
+    remote = "https://github.com/bazelbuild/rules_cc",
+)
 
 # Not all boost libs are available at the latest version yet, so use a
 # slightly older one that has all we need.
@@ -26,24 +36,25 @@ bazel_dep(name = "boost.bind", version = BOOST_VERSION)
 bazel_dep(name = "boost.config", version = BOOST_VERSION)
 bazel_dep(name = "boost.core", version = BOOST_VERSION)
 bazel_dep(name = "boost.fusion", version = BOOST_VERSION)
-bazel_dep(name = "boost.geometry", version = BOOST_VERSION)
-bazel_dep(name = "boost.graph", version = BOOST_VERSION)
+bazel_dep(name = "boost.geometry", version = BOOST_VERSION + ".bcr.2")
+bazel_dep(name = "boost.graph", version = BOOST_VERSION + ".bcr.2")
 bazel_dep(name = "boost.heap", version = BOOST_VERSION)
 bazel_dep(name = "boost.icl", version = BOOST_VERSION)
-bazel_dep(name = "boost.json", version = BOOST_VERSION)
-bazel_dep(name = "boost.lambda", version = BOOST_VERSION)
-bazel_dep(name = "boost.multi_array", version = BOOST_VERSION)
+bazel_dep(name = "boost.json", version = BOOST_VERSION + ".bcr.2")
+bazel_dep(name = "boost.lambda", version = BOOST_VERSION + ".bcr.2")
+bazel_dep(name = "boost.multi_array", version = BOOST_VERSION + ".bcr.2")
 bazel_dep(name = "boost.optional", version = BOOST_VERSION)
 bazel_dep(name = "boost.phoenix", version = BOOST_VERSION)
-bazel_dep(name = "boost.polygon", version = BOOST_VERSION)
-bazel_dep(name = "boost.property_tree", version = BOOST_VERSION)
+bazel_dep(name = "boost.polygon", version = BOOST_VERSION + ".bcr.2")
+bazel_dep(name = "boost.property_tree", version = BOOST_VERSION + ".bcr.2")
 bazel_dep(name = "boost.spirit", version = BOOST_VERSION)
 bazel_dep(name = "boost.stacktrace", version = BOOST_VERSION)
 bazel_dep(name = "boost.thread", version = BOOST_VERSION)
 bazel_dep(name = "cudd", version = "3.0.0")
 bazel_dep(name = "eigen", version = "3.4.0.bcr.2")
 bazel_dep(name = "or-tools", version = "9.12")
 bazel_dep(name = "spdlog", version = "1.15.1")
+bazel_dep(name = "tcmalloc", version = "0.0.0-20250331-43fcf6e")
 bazel_dep(name = "zlib", version = "1.3.1.bcr.5")
 
 bazel_dep(name = "googletest", version = "1.16.0", dev_dependency = True)
@@ -63,4 +74,5 @@ llvm.toolchain(
 use_repo(llvm, "llvm_toolchain")
 # use_repo(llvm, "llvm_toolchain_llvm") # if you depend on specific tools in scripts
 
+# FYI: Comment out on NixOS where you'd like to use the local clang toolchain.
 register_toolchains("@llvm_toolchain//:all")