summaryrefslogtreecommitdiff
path: root/core/src
diff options
context:
space:
mode:
authorJack Koenig2021-06-29 16:39:45 -0700
committerJack Koenig2021-06-29 16:55:15 -0700
commit25a84b5667614ea3f437b656f1939caba57e6f66 (patch)
treed5dbd592ceed3040209bb74dfe1928b983052c08 /core/src
parent0531cb53d3cedaff33c2a280e34418f6af5bc6a1 (diff)
Change behavior of aop.Select to not include CloneModuleAsRecord
Previously, CloneModuleAsRecord clones would result in the same BaseModule object coming up multiple times when using APIs like .instances, .collectDeep, and .getDeep. This was not the intended behavior and can lead to very subtle bugs.
Diffstat (limited to 'core/src')
-rw-r--r--core/src/main/scala/chisel3/Module.scala2
1 files changed, 1 insertions, 1 deletions
diff --git a/core/src/main/scala/chisel3/Module.scala b/core/src/main/scala/chisel3/Module.scala
index f82b6a7b..41fe4554 100644
--- a/core/src/main/scala/chisel3/Module.scala
+++ b/core/src/main/scala/chisel3/Module.scala
@@ -184,7 +184,7 @@ package internal {
object BaseModule {
// Private internal class to serve as a _parent for Data in cloned ports
- private[chisel3] class ModuleClone(private[chisel3] val _proto: BaseModule) extends BaseModule {
+ private[chisel3] class ModuleClone(_proto: BaseModule) extends BaseModule {
// ClonePorts that hold the bound ports for this module
// Used for setting the refs of both this module and the Record
private[BaseModule] var _portsRecord: Record = _