#1261 Further support for constant functions (fixes br920)

closed
nobody
None
5
2013-02-04
2013-02-03
Martin Whitaker
No

This patch implements the evaluate_function method for the NetEBBits, NetEBDiv, NetEBLogic, NetEBMinMax, and NetEBPow classes. It also factors out some common code into the NetEBinary class.

1 Attachments

Discussion

  • Here is a regression test for the test suite. This is expected to fail at compile time with V0.9.

    N.B. I had to remove the tests for NetEBMinMax, because tgt-vvp doesn't support the min/max operators.

     
    Last edit: Martin Whitaker 2013-02-03
    Attachments
  • Here is a regression test for the unary operators. It needs to be compiled with the -gverilog-ams flag. Again, it is expected to fail at compile time with V0.9.

     
    Attachments
  • This patch just factors out some common code in the various NetEBinary and NetEUnary sub-classes. Apply it if you think this is a good idea, throw it away if you don't! Note it does need to be applied after the above two patches.

     
  • Cary R.
    Cary R.
    2013-02-04

    I have applied these patches and pushed the test suite changes. I guess we need to look at the NetEBMinMAx stuff. I thought tgt-vvp supported all that.

     
  • Cary R.
    Cary R.
    2013-02-04

    • status: open --> closed