lowRISC / lowRISC/style-guides

Check the return value of the predict method

Open
#81 2 comments 0 reactions 1 assignee Claimed by @antmarzam View on GitHub
guideline suggestion
Dominant language
No language data
Stars
541
Forks
132
PR merge metrics
No merged PRs in 30d

Description

Do not cast the uvm_reg predict method to void, instead check the return value.

Argument:
The predict method may fail depending on whether it uses UVM_PREDICT_READ, UVM_PREDICT_WRITE or UVM_PREDICT_DIRECT, and ignoring the return value effectively hides whether the prediction was successful or not.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.