Merge pull request #10493 from The-OpenROAD-Project-staging/syn-fix-assert

povik · web-flow · commit 978be9b3cf45 · 2026-05-22T17:29:35.000+02:00
syn: fix stale assert
diff --git a/src/syn/src/elab/backend_builder.h b/src/syn/src/elab/backend_builder.h
@@ -49,8 +49,7 @@ class BackendGraphBuilder : public BackendGraphBuilderBase
   // is no longer called.
   syn::Graph& graph()
   {
-    assert(graph_.has_value());
-    // NOLINTNEXTLINE(bugprone-unchecked-optional-access)
+    assert(graph_);
     return *graph_;
   }
 

Original file line number	Diff line number	Diff line change
`@@ -49,8 +49,7 @@ class BackendGraphBuilder : public BackendGraphBuilderBase`
`49`	`49`	`// is no longer called.`
`50`	`50`	`syn::Graph& graph()`
`51`	`51`	`{`
`52`		`- assert(graph_.has_value());`
`53`		`- // NOLINTNEXTLINE(bugprone-unchecked-optional-access)`
	`52`	`+ assert(graph_);`
`54`	`53`	`return *graph_;`
`55`	`54`	`}`
`56`	`55`