You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tvm.apache.org by Kevin Laeufer <no...@github.com.INVALID> on 2022/01/12 22:10:00 UTC

[apache/tvm-vta] Port to new Chisel stable release (3.5) (PR #37)

This PR ports the VTA hardware to use the recently released new stable version of Chisel. It also fixes all new warnings.
The tests were all ported to use the `chiseltest` library since the `chisel-iotesters` are now deprecated. I did remove the `TestRunner` class since it did not seem to be used and in general we recommend to use `sbt test` or `sbt testOnly` to run unittests instead of writing a custom runner.
You can view, comment on, or merge this pull request online at:

  https://github.com/apache/tvm-vta/pull/37

-- Commit Summary --

  * upgrade to stable chisel 3.5.0
  * fix chiesl 3.5 warnings
  * port tests to chiseltest

-- File Changes --

    M hardware/chisel/build.sbt (11)
    M hardware/chisel/src/main/scala/core/FetchVME64.scala (2)
    M hardware/chisel/src/main/scala/core/FetchWideVME.scala (22)
    M hardware/chisel/src/main/scala/core/LoadUop.scala (2)
    M hardware/chisel/src/main/scala/core/LoadUopSimple.scala (12)
    M hardware/chisel/src/main/scala/core/TensorAlu.scala (8)
    M hardware/chisel/src/main/scala/core/TensorLoadNarrowVME.scala (20)
    M hardware/chisel/src/main/scala/core/TensorLoadSimple.scala (32)
    M hardware/chisel/src/main/scala/core/TensorLoadWideVME.scala (14)
    M hardware/chisel/src/main/scala/core/TensorStoreNarrowVME.scala (12)
    M hardware/chisel/src/main/scala/core/TensorStoreWideVME.scala (14)
    M hardware/chisel/src/main/scala/core/TensorUtil.scala (8)
    M hardware/chisel/src/main/scala/dpi/VTAHostDPI.scala (4)
    M hardware/chisel/src/main/scala/dpi/VTAMemDPI.scala (4)
    M hardware/chisel/src/main/scala/shell/SimShell.scala (4)
    M hardware/chisel/src/main/scala/shell/VCR.scala (12)
    M hardware/chisel/src/main/scala/shell/VME.scala (23)
    M hardware/chisel/src/main/scala/shell/VMESimple.scala (14)
    M hardware/chisel/src/main/scala/test/Test.scala (2)
    M hardware/chisel/src/main/scala/util/GenericParameterizedBundle.scala (20)
    M hardware/chisel/src/main/scala/util/SyncQueue.scala (52)
    M hardware/chisel/src/test/scala/unittest/AluTest.scala (2)
    M hardware/chisel/src/test/scala/unittest/GemmTest.scala (4)
    M hardware/chisel/src/test/scala/unittest/Generic.scala (24)
    D hardware/chisel/src/test/scala/unittest/Launcher.scala (60)
    M hardware/chisel/src/test/scala/unittest/MvmTest.scala (3)
    M hardware/chisel/src/test/scala/unittest/SyncQueue2PortMemTest.scala (4)
    M hardware/chisel/src/test/scala/unittest/SyncQueueTest.scala (3)
    M hardware/chisel/src/test/scala/unittest/TensorAluTest.scala (3)
    M hardware/chisel/src/test/scala/unittest/TensorGemmJsonTest.scala (3)
    M hardware/chisel/src/test/scala/unittest/TensorGemmTest.scala (3)
    D hardware/chisel/src/test/scala/unittest/utils/TestRunner.scala (91)

-- Patch Links --

https://github.com/apache/tvm-vta/pull/37.patch
https://github.com/apache/tvm-vta/pull/37.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/apache/tvm-vta/pull/37
You are receiving this because you are subscribed to this thread.

Message ID: &lt;apache/tvm-vta/pull/37@github.com&gt;

Re: [apache/tvm-vta] Port to new Chisel stable release (3.5) (PR #37)

Posted by Kevin Laeufer <no...@github.com.INVALID>.
@vegaluisjose Anything I should do/change to get this merged?

-- 
Reply to this email directly or view it on GitHub:
https://github.com/apache/tvm-vta/pull/37#issuecomment-1023598716
You are receiving this because you are subscribed to this thread.

Message ID: <ap...@github.com>

Re: [apache/tvm-vta] Port to new Chisel stable release (3.5) (PR #37)

Posted by Thierry Moreau <no...@github.com.INVALID>.
Merged #37 into main.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/apache/tvm-vta/pull/37#event-5966816624
You are receiving this because you are subscribed to this thread.

Message ID: <ap...@github.com>