mpeg5 / xevd

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

Problem building with -DARM=TRUE #56

Closed wader closed 5 months ago

wader commented 6 months ago

Hello! i'm trying build with -DARM=TRUE with alpine linux via docker on an aarch64 host (mac book m3) but running into various build errors.

$ uname -a
Linux 00823b109f1f 6.6.16-linuxkit #1 SMP Fri Feb 16 11:54:02 UTC 2024 aarch64 Linux

$ mkdir build && cd build
$ cmake -DARM=TRUE ..
$ gcc --version
gcc (Alpine 13.2.1_git20231014) 13.2.1 20231014
Copyright (C) 2023 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Some problems encountered:

/xevd/src_main/xevdm.c:3386:28: error: 'xevdm_itrans_map_tbl_neon' undeclared (first use in this function); did you mean 'xevdm_itrans_map_tbl'?
 3386 |     xevd_func_itrans     = xevdm_itrans_map_tbl_neon;
/xevd/src_main/neon/xevdm_itdq_neon.c:1401:14: error: array subscript 16 is above array bounds of 'int32x4_t[16]' [-Werror=array-bounds=]
 1401 |             t[16] = vzip2q_s32(v[0], v[1]);
/xevd/src_main/neon/xevdm_mc_neon.c: In function 'xevdm_bl_mc_l_n0_neon':
/xevd/src_main/neon/xevdm_mc_neon.c:827:57: error: 'tbl_bl_mc_l_coeff' undeclared (first use in this function); did you mean 'tbl_mc_l_coeff'?
  827 |     mc_filter_bilin_horz_neon(ref, s_ref, pred, s_pred, tbl_bl_mc_l_coeff[dx], w, h, min, max, MAC_ADD_N0, MAC_SFT_N0, 1);
      |                                                         ^~~~~~~~~~~~~~~~~
      |                                                         tbl_mc_l_coeff
dariusz-f commented 6 months ago

Check if #57 fixes this problem

wader commented 5 months ago

I think we can close this one now? but i did run into a similar amd64 issue now see https://github.com/mpeg5/xevd/pull/57#issuecomment-2068213948

wader commented 5 months ago

Thanks for fix and help!