oneapi-src / oneAPI-samples

Samples for Intel® oneAPI Toolkits
https://oneapi-src.github.io/oneAPI-samples/
MIT License
922 stars 681 forks source link

Simplify niosv sim #2464

Closed whitepau closed 3 weeks ago

whitepau commented 3 weeks ago

Existing Sample Changes

Description

An RTL expert advised removing sim:/ from the following lines of code in wave.do:

https://github.com/oneapi-src/oneAPI-samples/blob/2ed1542de0245e426e3ba1fef81afa66209f3070/DirectProgramming/C%2B%2BSYCL_FPGA/ReferenceDesigns/niosv/simulation_files/wave.do#L1-L3

add wave -group "Nios V" -radix hexadecimal -position insertpoint sim:/pd_system_tb/pd_system_inst/nios/*

add wave -group "Code & Data RAM" -radix hexadecimal -position insertpoint sim:/pd_system_tb/pd_system_inst/code_data_ram/*

add wave -group "Accelerator" -radix hexadecimal -position insertpoint sim:/pd_system_tb/pd_system_inst/simple_dma_accelerator/*

I also combined some of the .gitignore files and made a small readme change. None of these affected the source code that is tested, but I ran a regtest as a sanity check:

https://spetc.intel.com/testanalysis?trview=0&testRunIds=10618192&tapgsize=20

Fixes Issue# https://hsdes.intel.com/appstore/article/#/18039676682

External Dependencies

List any external dependencies created as a result of this change.

Type of change

Please delete options that are not relevant. Add a 'X' to the one that is applicable.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration