freechipsproject / firrtl-interpreter

A scala based simulator for circuits described by a LoFirrtl file
Other
46 stars 23 forks source link

DriverCompatibility => DriverCompatibilityExtensions #223

Closed jackkoenig closed 2 years ago

jackkoenig commented 2 years ago

DriverCompatibility still exists in firrtl so we want to ensure there are no classpath collisions. Instead, we'll represent polyfill methods via extension methods. This does require an additional import (it's not source compatible), but it's the best we can do.

Needed to make https://github.com/freechipsproject/chisel-testers/pull/325 work

jackkoenig commented 2 years ago

The diff looks large, but that's because it includes lots of stuff that still exists in the version of DriverCompatibility that lives in firrtl. I removed all of that unnecessary stuff and made this just provide the function that had been deleted from firrtl.