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 合并指标
 (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.

贡献者指南