Open emacdo12 opened 4 years ago
this is odd, we have test that work for this form, only hierarchical overrides are not yet supported
@hmoein87 Would you please have a look at this?
@HMoein87 : @ArashAhmadian is one of UoT students working on improving the regression test. He has recently faced the same issue you have been working on last semester. In 64-16bit-fixed-JACM.v.txt, Odin_II throws an error at line 1057. When we use a constant value, like 10, instead of depth at line 1057, Odin_II continues working properly. Please provide an appropriate solution to this aspect of the issue for him. Moreover, please update this branch and create a new PR in the main VTR repository so that we shall merge it.
seems the code is not complete. it has some declaration errors.
Expected Behaviour
If the proper syntax is in place, it is possible to override the parameters in a seperate model with new values. There are two ways to do it, defparam and model #(.paramter(new_parameter)) model (ports[]); I have included test cases for both.
Current Behaviour
Odin_II doesn't do the overried (defparam) or can't handle it and aborts.
Possible Solution
Steps to Reproduce
All these cases have been verified through quartus.
defparam:
In this last test case you can see that where the output vector differs is the most significant bit indicating that the defparam didn't motify the parameters of the assg model. If it had, the msb would have been passed through.
Context
Your Environment