Skip to content
Snippets Groups Projects

upd: remove many warnings at compile time in release mode

Merged 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
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading