From e4cbde18e10aa1d672e2edd853673fb27097728b Mon Sep 17 00:00:00 2001
From: Miodrag Milanovic <mmicko@gmail.com>
Date: Mon, 18 May 2026 09:44:57 +0200
Subject: [PATCH] Fixed warnings found by gcc-16

---
 backends/jny/jny.cc     | 2 +-
 passes/cmds/abstract.cc | 1 +
 passes/cmds/sdc/sdc.cc  | 2 +-
 passes/opt/opt_merge.cc | 2 +-
 4 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/backends/jny/jny.cc b/backends/jny/jny.cc
index ee0c0d14c..3f087d260 100644
--- a/backends/jny/jny.cc
+++ b/backends/jny/jny.cc
@@ -275,7 +275,7 @@ struct JnyWriter
         const auto _indent = gen_indent(indent_level);
 
         bool first_port{true};
-        for (auto con : port_cell->connections()) {
+        for (auto& con : port_cell->connections()) {
             if (!first_port)
                 f << ",\n";
 
diff --git a/passes/cmds/abstract.cc b/passes/cmds/abstract.cc
index 2519daf5c..698907eb9 100644
--- a/passes/cmds/abstract.cc
+++ b/passes/cmds/abstract.cc
@@ -469,6 +469,7 @@ struct AbstractPass : public Pass {
 			switch (enable) {
 				case Enable::Always:
 					log_assert(false);
+					YS_FALLTHROUGH
 				case Enable::ActiveLow:
 				case Enable::ActiveHigh: {
 					if (enable_name.empty())
diff --git a/passes/cmds/sdc/sdc.cc b/passes/cmds/sdc/sdc.cc
index cb0b074d5..400d9693c 100644
--- a/passes/cmds/sdc/sdc.cc
+++ b/passes/cmds/sdc/sdc.cc
@@ -148,7 +148,7 @@ struct SdcObjects {
 				path += "/";
 			path += name;
 			design_cells.push_back(std::make_pair(path, cell));
-			for (auto pin : cell->connections()) {
+			for (auto& pin : cell->connections()) {
 				IdString pin_name = pin.first;
 				std::string pin_name_sdc = path + "/" + pin.first.str().substr(1);
 				design_pins.push_back(std::make_pair(pin_name_sdc, std::make_pair(cell, pin_name)));
diff --git a/passes/opt/opt_merge.cc b/passes/opt/opt_merge.cc
index a6121b268..003cfda37 100644
--- a/passes/opt/opt_merge.cc
+++ b/passes/opt/opt_merge.cc
@@ -324,7 +324,7 @@ struct OptMergeThreadWorker
 			CellEqualOp> known_cells(0, CellHashOp(), CellEqualOp(*this));
 
 		std::vector<DuplicateCell> duplicates;
-		for (const std::vector<std::vector<CellHash>> &buckets : in.bucketed_cell_hashes) {
+		for (std::vector<std::vector<CellHash>> &buckets : in.bucketed_cell_hashes) {
 			// Clear out our buckets as we go. This keeps the work of deallocation
 			// off the main thread.
 			std::vector<CellHash> bucket = std::move(buckets[index]);