Jalali
Jalali
Hello @silabs-robin , thanks for your feedback, I'm with the idea of removing default values, because could be assigning wrong value because of a copy-paste bug, and I would like...
@silabs-robin I notice this problem with the toggle coverage with C extension because of the filed length in the compressed instruction. Thank you that was helpful, and if i had...
@MikeOpenHWGroup @silabs-robin ,if i'm not wrong, there is more of this copy-paste bug, please look at : https://github.com/openhwgroup/core-v-verif/blob/05ba4eb0cf6296bf35f487fbe2ab01e532d0fe26/lib/uvm_agents/uvma_isacov/cov/uvma_isacov_cov_model.sv#L1756 https://github.com/openhwgroup/core-v-verif/blob/05ba4eb0cf6296bf35f487fbe2ab01e532d0fe26/lib/uvm_agents/uvma_isacov/cov/uvma_isacov_cov_model.sv#L1687 it's the same value for ADDI, but we should correct rs1...
@MikeOpenHWGroup on the master branch ?
@MikeOpenHWGroup here is the PR https://github.com/openhwgroup/core-v-verif/pull/1405
@silabs-robin, I totally agree with you, so the solution is to use the c_has_rs1, because the c.mv & c.add share the same format, so we could remove c.mv from c_has_rs1...
@MikeOpenHWGroup Sure !!
@MikeOpenHWGroup PR #1442 to fix the issue
@silabs-robin, think you for this, i think it will be very helpful for me. Answering your question , I was investigating some missing coverage (bins related to value type), and...
hello, I prefer also to simply decode instructions, it difficult a little bit but it's simple to read instead of the **dpi_dasm**. Yes also for CIW, i just give an...