mpeg5 / xevd

eXtra-fast Essential Video Decoder, MPEG-5 EVC (Essential Video Coding)
Other
71 stars 22 forks source link

Warning in GCC 11, 12 #3

Closed Jamaika1 closed 3 years ago

Jamaika1 commented 3 years ago
    inlined from 'xevd_recon_tree' at xevdm.c:1778:9:
xevdm.c:1124:9: warning: 'xevdm_IBC_mc' accessing 98304 bytes in a region of size 32768 [-Wstringop-overflow=]
 1124 |         xevdm_IBC_mc(x, y, log2_cuw, log2_cuh, core->mv[0], ctx->pic, core->pred[0], mcore->tree_cons, ctx->sps.chroma_format_idc);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
xevdm.c: In function 'xevd_recon_tree':
xevdm.c:1124:9: note: referencing argument 7 of type 'pel (*)[16384]' {aka 'short int (*)[16384]'}
In file included from xevdm_def.h:625,
                 from xevdm.c:38:
xevdm_mc.h:76:6: note: in a call to function 'xevdm_IBC_mc'
   76 | void xevdm_IBC_mc(int x, int y, int log2_cuw, int log2_cuh, s16 mv[MV_D], XEVD_PIC *ref_pic, pel pred[N_C][MAX_CU_DIM], TREE_CONS tree_cons, int chroma_format_idc);
      |      ^~~~~~~~~~~~
mpeg5 commented 3 years ago

@mss-park Could you check this issue?

mss-park commented 3 years ago

The problem of gcc-11 had been solved in PR #5. Please check current master code.

Jamaika1 commented 3 years ago

Hi, I test b6ff17a in GCC 12.0.0 20210722

xevdm.c: In function 'xevdm_platform_init':
xevdm.c:3270:30: warning: assignment to 'void (* const (*)[6])(void *, void *, int,  int,  int)' from incompatible pointer type 'void (* const*)(void *, void *, int,  int,  int)' [-Wincompatible-pointer-types]
 3270 |         ctx->fn_itxb         = xevd_tbl_itxb;
      |                              ^
In function 'xevd_recon_unit',
    inlined from 'xevd_recon_tree' at xevdm.c:1792:9:
xevdm.c:1138:9: warning: 'xevdm_IBC_mc' accessing 98304 bytes in a region of size 32768 [-Wstringop-overflow=]
 1138 |         xevdm_IBC_mc(x, y, log2_cuw, log2_cuh, core->mv[0], ctx->pic, core->pred[0], mcore->tree_cons, ctx->sps.chroma_format_idc);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
xevdm.c:1138:9: note: referencing argument 7 of type 'pel (*)[16384]' {aka 'short int (*)[16384]'}
In file included from xevdm_def.h:616,
                 from xevdm.c:38:
xevdm_mc.h: In function 'xevd_recon_tree':
xevdm_mc.h:76:6: note: in a call to function 'xevdm_IBC_mc'
   76 | void xevdm_IBC_mc(int x, int y, int log2_cuw, int log2_cuh, s16 mv[MV_D], XEVD_PIC *ref_pic, pel pred[N_C][MAX_CU_DIM], TREE_CONS tree_cons, int chroma_format_idc);
mpeg5 commented 3 years ago

@mss-park Please check the above warning message and update code again.

mpeg5 commented 3 years ago

@Jamaika1

I belive this issue has sorted out, now. Check the the latest PR code (783ff40ff2b1e1504d0c136d3b539085ffb9ff7f) again and let us know.

Jamaika1 commented 3 years ago

Thanks for corrections. There is no warming message