Skip to content

Conversation

@jinboson
Copy link
Contributor

No description provided.

With CFLAGS=-Werror, gcc14 outputs errors like below:

In file included from ./codec/common/inc/loongson_intrinsics.h:63,
                 from codec/common/loongarch/mc_horver_lsx.c:46:
In function ‘__lsx_vilvl_h’,
    inlined from ‘McHorVer22WidthEq4_lsx’ at codec/common/loongarch/mc_horver_lsx.c:521:5,
    inlined from ‘McHorVer22WidthEq9_lsx’ at codec/common/loongarch/mc_horver_lsx.c:646:3:
/usr/lib/gcc-cross/loongarch64-linux-gnu/14/include/lsxintrin.h:1916:19: warning: ‘dst6’ may be used uninitialized [-Wmaybe-uninitialized]
 1916 |   return (__m128i)__builtin_lsx_vilvl_h ((v8i16)_1, (v8i16)_2);
      |                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
codec/common/loongarch/mc_horver_lsx.c: In function ‘McHorVer22WidthEq9_lsx’:
codec/common/loongarch/mc_horver_lsx.c:486:47: note: ‘dst6’ was declared here
  486 |   __m128i dst0, dst1, dst2, dst3, dst4, dst5, dst6, dst7;
      |                                               ^~~~

Note: errors are truncated because too long.
@jinboson
Copy link
Contributor Author

sudo apt-get install -qq gcc-14-loongarch64-linux-gnu g++-14-loongarch64-linux-gnu qemu-user-static

The above cmd works fine in my fork repository https://github.com/jinboson/openh264-loongson/actions/runs/12707266185/job/35421827233n, but it fails here, why ?

@GuangweiWang
Copy link
Collaborator

sudo apt-get install -qq gcc-14-loongarch64-linux-gnu g++-14-loongarch64-linux-gnu qemu-user-static

The above cmd works fine in my fork repository https://github.com/jinboson/openh264-loongson/actions/runs/12707266185/job/35421827233n, but it fails here, why ?

please refer to following error:
E: Couldn't find any package by regex 'g++-14-loongarch64-linux-gnu'

any additional action to support loogarch64 ?

@jinboson
Copy link
Contributor Author

jinboson commented Jan 13, 2025

sudo apt-get install -qq gcc-14-loongarch64-linux-gnu g++-14-loongarch64-linux-gnu qemu-user-static

After check we found that the above cmd needs to run on ubuntu-24.04, thank you @GuangweiWang .

@GuangweiWang
Copy link
Collaborator

sudo apt-get install -qq gcc-14-loongarch64-linux-gnu g++-14-loongarch64-linux-gnu qemu-user-static

After check we found that the above cmd needs to run on ubuntu-24.04, thank you @GuangweiWang .

CC: @BenzhengZhang

@BenzhengZhang
Copy link
Collaborator

@GuangweiWang Long time no see 😭

@BenzhengZhang BenzhengZhang merged commit e49c095 into cisco:master Jan 13, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants