Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
#0: Remove incorrect memory span assert (#16136)
### Ticket n/A ### Problem description This assert was added when kernel data packing was implemented (data load address immediately after text). But that only worked for (non-idle) erisc kernels by accident due to another bug an obsolete workaround. I fixed this with ``` * 434bd8e 2024-12-13 | #13944: Redesign memory packing API (#15980) ``` By not packing such erisc kernels. This worked in production builds because asserts are disabled, so I didn't fall over this problem. ### What's changed Remove asserts, update comments to reflect reality. working on updating non-idle erisc to allow packing (perhaps CI optimized builds should enable asserts, Remember CMAKE_BUILD_TYPE=RelWithDebInfo doesn't do that) ### Checklist - [Yes] Post commit CI passes - [ ] Blackhole Post commit (if applicable) - [ ] Model regression CI testing passes (if applicable) - [ ] Device performance regression CI testing passes (if applicable) - [ ] New/Existing tests provide coverage for changes
- Loading branch information