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
- the bitstream packing
- defines not recognized by the WMC tool, e.g. FL2WORD16()
- 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?