lowRISC / lowRISC/opentitan

[prim_reg_cdc, CDC boundary] RTL simplification to avoid CCOV unreachable gap

Open
#26,164 0 comments 0 reactions 1 assignee Claimed by @antmarzam View on GitHub
Component:CDC Component:RTL IP:prim Type:FutureRelease
Dominant language
SystemVerilog
Stars
3.6k
Forks
1.1k
Avg merge
2d 22h
Merged PRs (30d)
141

Description

### Description

Prim_reg_cdc has some conditional coverage which looks unhittable and it would be helpful to have a simpler condition to close coverage rather than having an waiver exclusion for each instance.

The reasoning of why this is not reachable can be found [here](https://github.com/lowRISC/opentitan/pull/25705#discussion_r1927318143)

There's an [outstanding PR](https://github.com/lowRISC/opentitan/pull/25812) aiming to simplify the condition. As of now the PR won't be merged until we do the next CDC sign-off.

Action: revisit this issue once we tackle CDC sign-off again.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.