Reverted the last commit, noting that the implementation physically copies the dnwell and subcut layers up the hierarchy, which is not a proper use of "copyup", as it creates unnecessary redundant layout that, in theory, should have no impact on the extraction. This may have at one time worked around an issue in magic's extractor. If there is still an extraction issue, then the problem needs to be fixed in magic, not with a workaround. If really needed, it should be recast as an extraction variant so as not to interfere with normal use of the extractor.
diff --git a/VERSION b/VERSION index fa8116d..4cb34c1 100644 --- a/VERSION +++ b/VERSION
@@ -1 +1 @@ -1.0.329 +1.0.330
diff --git a/sky130/magic/sky130.tech b/sky130/magic/sky130.tech index 299f9e9..76e92f8 100644 --- a/sky130/magic/sky130.tech +++ b/sky130/magic/sky130.tech
@@ -2460,17 +2460,10 @@ variants * labels SUBPIN port - templayer alldnwell DNWELL,lowerdnwell - copyup lowerdnwell - - layer dnwell DNWELL,lowerdnwell + layer dnwell DNWELL labels DNWELL - templayer alliso SUBCUT,loweriso - copyup loweriso - - layer isosub SUBCUT,loweriso - and-not DNWELL,lowerdnwell + layer isosub SUBCUT labels SUBCUT layer npn DNWELL