upd: remove many warnings at compile time in release mode
Context
Release compilation suffered from hundreds of warnings. This MR solves most of them.
Main changes
- move some inline functions from header to source
- include what is used in header and source files to avoid relying on recursive includes
- remove 'constexpr' from 'Tensor(Vector)' constructor that applied forced inlining
- use C++ types and not C type (std::size_t instead of size_t)
- move some default constructor/operators to src
- explicitly delete some default class constructors for 'SingleGraphMatching', 'StaticAnalysis', 'MetaOpStats' and 'OperatorStats'
- add default parameters to 'ArgMax' constructor
- make Node::mAttrs available to the associated Operator instead of accessing it each time getOperator() is called
- make some for loop access elements by reference in 'FuseBatchNorm'
Edited by Maxence Naud
Merge request reports
Activity
Filter activity
changed milestone to %aidge v0.5.0
added Enhancement ⭐ label
assigned to @pineapple
added 1 commit
- 0ab6f45d - add noexcept keyword to 'Context' destructor
added 7 commits
-
0ab6f45d...9dd41e85 - 4 commits from branch
dev
- 2686e346 - upd: remove many warnings at compile time in release mode
- b7ea51c4 - add small changes
- 3357d08c - add noexcept keyword to 'Context' destructor
Toggle commit list-
0ab6f45d...9dd41e85 - 4 commits from branch
enabled an automatic merge when all merge checks for 3357d08c pass
mentioned in commit 434e21e3
Please register or sign in to reply