[libre-riscv-dev] [Bug 316] bperm TODO

[bugzilla-daemon at libre-soc.org]

https://bugs.libre-soc.org/show_bug.cgi?id=316

--- Comment #52 from Cole Poirier <colepoirier at gmail.com> ---
Michael, or Luke,

In order to add Bpermd to LogicalMainStage, it appears that I first have to add
it to soc.decoder.power_enums.InternalOp, because that is the type of the cases
for the switch statement. The switch-case statment 'matches' on 'op.isn_type'
which is an InternalOp. Is it possible to user Bpermd inside of a with m.Case
statement, given that all the other m.Case statements wrap InternalOp.isn's and
the bpermd TODO comment template expects InternalOp.OP_BPERM? If I need to add
OP_BPERM to soc.decoder.power_enums.InternalOp, are there any special
instructions or restrictions for doing so? As this will be my first time
touching code that is used by other people I want to make sure I don't
interfere with or break anything.

-- 
You are receiving this mail because:
You are on the CC list for the bug.