Skip to content

Add qemu test for LoongArch#3833

Merged
BenzhengZhang merged 2 commits intocisco:masterfrom
jinboson:master
Jan 13, 2025
Merged

Add qemu test for LoongArch#3833
BenzhengZhang merged 2 commits intocisco:masterfrom
jinboson:master

Conversation

@jinboson
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Collaborator

@GuangweiWang Long time no see 😭

@BenzhengZhang BenzhengZhang merged commit e49c095 into cisco:master Jan 13, 2025
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