circt icon indicating copy to clipboard operation
circt copied to clipboard

Published 20 hours ago • verified publisher icon llvm

[FIRRTL] Mux reduction

Open darthscsi opened this issue 2 years ago • 3 comments

This reduces multi-muxes with certain patterns of redundancy. This is a very abbreviated version of prime-implicant style coverings.

This can change the output value for an out-of-bounds index,

darthscsi avatar Nov 10 '23 21:11 darthscsi

Also, having index == 0 being input[last] is supper supper annoying.

darthscsi avatar Nov 10 '23 21:11 darthscsi

Also, having index == 0 being input[last] is supper supper annoying.

I agree it's really annoying but this is consistent withfirrtl.mux and hw.array_create.

uenoku avatar Nov 11 '23 03:11 uenoku

Also, having index == 0 being input[last] is supper supper annoying.

I agree it's really annoying but this is consistent withfirrtl.mux and hw.array_create.

That doesn't mean we got those right either :)

darthscsi avatar Nov 13 '23 16:11 darthscsi