WMC_TOOL_SKIP instances should be reduced

The number of WMC_TOOL_SKIP instances should be reduced. The macro is currently used at places related to

  1. the bitstream packing
  2. defines not recognized by the WMC tool, e.g. FL2WORD16()
  3. around BASOP code in the FLP framework

Concerning the point 3,

  • I am not sure whether it is still relevant as the WMC tool instruments and counts the BASOP operators as well.
  • It is likely not relevant in the BASOP framework and should be removed from these places, see e.g. https://forge.3gpp.org/rep/sa4/audio/ivas-basop/-/blob/main/lib_enc/hq_lr_enc_fx.c#L1348 or https://forge.3gpp.org/rep/sa4/audio/ivas-basop/-/blob/main/lib_dec/arith_coder_dec_fx.c#L359

@malenovskyvl, could you clarify it, please?

Assignee Loading
Time tracking Loading