ucb-bar/onnxruntime-riscv

Don't require full rebuild when doing --for_firesim

開放

#30 建立於 2021年3月21日

 (2 則留言) (0 個反應) (0 位負責人)C++ (37 個分叉)auto 404
good first issue

倉庫指標

星標
 (93 顆星)
PR 合併指標
 (30 天內沒有已合併 PR)

描述

Whenever we add on --for_firesim this forces a full rebuild. It's very annoying and slows down development speed. This should not be required because all for_firesim on the ORT binary does is basically flush gemmini at process start (the flag for the runner does mlockall, but since runner builds in 1 min anyway that's not an issue).

I propose instead of controller whether flush happens via an ifdef, we just always flush on the first call to gemmini if the execution mode is WS. Basically have a global variable that stores whether flush has happened, and then on the very first call to one of the gemmini functions (conv/matmul) we flush if mode is WS. The get_matmul_type function is a good place for this because all calls go through that.

貢獻者指南