Don't Dedup modules if it would change semantics (bp #1713) (#1714)

* Don't Dedup modules if it would change semantics (#1713)

If a module has ports of type Bundle that are used in aggregate
connections in parent modules, Dedup cannot change the names of the
fields of the Bundle or it would change the semantics of the connection.
Dedup now detects this case and refrains from agnostifying the ports of
such modules to prevent this issue.

(cherry picked from commit e0e68568

)

# Conflicts:
#	src/main/scala/firrtl/transforms/Dedup.scala
#	src/test/scala/firrtlTests/transforms/DedupTests.scala

* Fixup for 1.2.x

Co-authored-by: Jack Koenig <koenig@sifive.com>