Merge pull request #1883 from KLayout/issue-1869

Fixed issue #1869 by adding a new option, -to or --top-output to strmxor
KLayout · Oct 20, 2024 · 6c82177 · 6c82177
2 parents 7c13ab1 + b4f3813
commit 6c82177
Show file tree

Hide file tree

Showing 15 changed files with 6 additions and 2 deletions.
diff --git a/src/buddies/src/bd/strmxor.cc b/src/buddies/src/bd/strmxor.cc
@@ -360,7 +360,7 @@ BD_PUBLIC int strmxor (int argc, char *argv[])
   generic_reader_options_b.set_group_prefix ("Input B");
 
   std::string infile_a, infile_b, output;
-  std::string top_a, top_b;
+  std::string top_a, top_b, top_output;
   bool dont_summarize_missing_layers = false;
   bool silent = false;
   bool no_summary = false;
@@ -393,6 +393,10 @@ BD_PUBLIC int strmxor (int argc, char *argv[])
       << tl::arg ("-tb|--top-b=name",          &top_b,      "Specifies the top cell for the second layout",
                   "See --top-a for details."
                  )
+      << tl::arg ("-to|--top-output=name",     &top_output, "Specifies the top cell for the output layout",
+                  "This option is only used if an output layout is given. It will specify the name of top cell to use there. "
+                  "If not specified, KLayout uses the top cell name of the first layout or the one given with --top-a."
+                 )
       << tl::arg ("-u|--deep",                 &deep,       "Deep (hierarchical mode)",
                   "Enables hierarchical XOR (experimental). In this mode, tiling is not supported "
                   "and the tiling arguments are ignored."
@@ -518,7 +522,7 @@ BD_PUBLIC int strmxor (int argc, char *argv[])
 
   if (! output.empty ()) {
     output_layout.reset (new db::Layout ());
-    output_top = output_layout->add_cell ("XOR");
+    output_top = output_layout->add_cell (top_output.empty () ? top_a.c_str () : top_output.c_str ());
   }
 
   std::map<std::pair<int, db::LayerProperties>, ResultDescriptor> results;

diff --git a/testdata/bd/strmxor_au1.oas b/testdata/bd/strmxor_au1.oas
diff --git a/testdata/bd/strmxor_au1d.oas b/testdata/bd/strmxor_au1d.oas
diff --git a/testdata/bd/strmxor_au2.oas b/testdata/bd/strmxor_au2.oas
diff --git a/testdata/bd/strmxor_au2d.oas b/testdata/bd/strmxor_au2d.oas
diff --git a/testdata/bd/strmxor_au3.oas b/testdata/bd/strmxor_au3.oas
diff --git a/testdata/bd/strmxor_au3_heal.oas b/testdata/bd/strmxor_au3_heal.oas
diff --git a/testdata/bd/strmxor_au3d.oas b/testdata/bd/strmxor_au3d.oas
diff --git a/testdata/bd/strmxor_au4.oas b/testdata/bd/strmxor_au4.oas
diff --git a/testdata/bd/strmxor_au4_heal.oas b/testdata/bd/strmxor_au4_heal.oas
diff --git a/testdata/bd/strmxor_au4d.oas b/testdata/bd/strmxor_au4d.oas
diff --git a/testdata/bd/strmxor_au5.oas b/testdata/bd/strmxor_au5.oas
diff --git a/testdata/bd/strmxor_au5d.oas b/testdata/bd/strmxor_au5d.oas
diff --git a/testdata/bd/strmxor_au6.oas b/testdata/bd/strmxor_au6.oas
diff --git a/testdata/bd/strmxor_au6d.oas b/testdata/bd/strmxor_au6d.oas