Closed leonardt closed 4 years ago
term is being defined later on (see https://github.com/rdaly525/coreir/blob/e070dc0c57a964fd65ed8b46a10eb9916b54a49a/include/coreir/definitions/corebitVerilog.hpp#L128-L135) so it should not be a part of vmap. The redefinition of term removes the primitive type because it overrides the vjson, but the expression lambda is still set from the earlier logic. Term shouldn't be part of vmap because it's body doesn't match the "assign =" pattern used for the other primitives.
vmap
vjson
"assign ="
term is being defined later on (see https://github.com/rdaly525/coreir/blob/e070dc0c57a964fd65ed8b46a10eb9916b54a49a/include/coreir/definitions/corebitVerilog.hpp#L128-L135) so it should not be a part of
vmap. The redefinition of term removes the primitive type because it overrides thevjson, but the expression lambda is still set from the earlier logic. Term shouldn't be part of vmap because it's body doesn't match the"assign ="pattern used for the other primitives.