rgiunti

Results 9 comments of rgiunti

Hi @emustafa96. I tested the PR making use of the UVM testbench [https://github.com/openhwgroup/cvfpu-uvm.git](url). In my test I set the FPU instance implementation in order to have merged slice for FMA...

Checked with random regression test. It can be merged

> Hi @rgiunti > > First of all thanks for taking interest in the scripts written by CEA. It is a very important upgrade of the scripts that you have...

> Another comment concerning the readme: I think there needs to be a section for post_proc command. Yes you're right. I'll update it

> Hi @rgiunti, thanks for your interest in OpenHW! This PR introduces a significantly new and different method for compiling and running tests with the cv_dv_utils. The original contribution of...

Hi @MikeOpenHWGroup and @khandelwaltanuj, thank you both for your comments and suggestions. Regarding UVM_NONE and dumping I can align my modifications to your suggestions easily. For Bender, that was just...

> Thanks for the review @khandelwaltanuj. Great suggestion regarding the user documentation. > > Hi @rgiunti, I will approve & merge this in as soon as you add the requested...

> > Hi Mike! I added some documentation on Bender > > Thanks @rgiunti, looks good. > > Once the license headers are pushed in, I'll approve/merge this PR. I...

Hi @JeanRochCoulon, I re-opened the PR after having rebased it. This time I used the verible format command explained in CONTRIBUTING.md so the code in /core/include should be aligned with...