diff options
author | Johan Alfven <johan.alfven@arm.com> | 2023-03-24 09:57:58 +0100 |
---|---|---|
committer | Johan Alfven <johan.alfven@arm.com> | 2023-05-02 10:40:46 +0200 |
commit | 463f74b2ed65753811ddf7460e760aaf983ef5bb (patch) | |
tree | 48531d0b1da05773ea48e880763338374e6bca44 /SUPPORTED_OPS.md | |
parent | 107063615e46a9d9d0f07db9cf139ac136978c77 (diff) | |
download | ethos-u-vela-463f74b2ed65753811ddf7460e760aaf983ef5bb.tar.gz |
MLBEDSW-7443: Temporal mem usage is erroneously calculated
- The array allocated in get_temporal_memory_usage is too small
so the first error is that not all LiveRange elements are added
to the temporal mem usage. The second error happens due to that
use_fast_storage_for_feature_maps is correctly trying to update
the temporal mem usage array but an assert happens due to out of
bounds. The array is too small since the LiveRangeClass is
reporting the wrong end time because of some inconsistencies in
how the mark usage is done for subgraph tensors.
- The fix is to mark the tensors with the current_time value.
Also changed so that tenors are marked consistently in both
extract functions. This means that the end time value to use
in get_temporal_memory_usage is the current_time + 1.
- Also made a small update to avoid updating current_time twice
when handling subgraphs.
Change-Id: Ib7e3681e370e097e433acb235740dfd69fa3ce8b
Signed-off-by: Johan Alfven <johan.alfven@arm.com>
Diffstat (limited to 'SUPPORTED_OPS.md')
0 files changed, 0 insertions, 0 deletions