Hi,
Thanks for your reply.
On Tue, 24 Oct 2023 11:05:39 +0000 Vasyl Gello wrote:
> Hi Dandan,
>
> Please ensure you have arch-specific constants in include files fixed
like here:
https://github.com/xbmc/xbmc/commit/69611c8fa937800d1c5d7897e798d56d770d7d3c
>
> Your upstream PR touches only system info display.
The extra compiler ifdefs has been submitted in the previous pull-request.
Please see this link: https://github.com/xbmc/xbmc/pull/22062/files
Except for system info, other files needed to add architectural support
already contain loongarch definitions in kodi 2:20.2+dfsg-4.
root@localhost:/home/kodi/kodi-20.2+dfsg# grep "loongarch" -ri . |grep
-v .pc
./xbmc/utils/MathUtils.h: defined(__loongarch__) || \
./xbmc/cores/DllLoader/ldt_keeper.c: !defined(__loongarch__) && \
./xbmc/cores/DllLoader/DllLoader.h: !defined(__loongarch__) && \
./cmake/scripts/linux/ArchSetup.cmake: elseif(CPU MATCHES loongarch64)
./cmake/scripts/linux/ArchSetup.cmake: set(ARCH loongarch64)
thanks,
Dandan Zhang
