Skip to content

upd: remove many warnings at compile time in release mode

Maxence Naud requested to merge rm_warnings into dev

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

Loading