Skip to content

Add riscv32 CI checks - #391

Open
matetokodi wants to merge 1 commit into
Samsung:mainfrom
matetokodi:riscv32_ci
Open

Add riscv32 CI checks#391
matetokodi wants to merge 1 commit into
Samsung:mainfrom
matetokodi:riscv32_ci

Conversation

@matetokodi

Copy link
Copy Markdown
Contributor

No description provided.

@zherczeg zherczeg left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ksh8281

ksh8281 commented Dec 10, 2025

Copy link
Copy Markdown
Collaborator

How about to set allow_failure flags on risc-v checks while fix risc-v issues?

@matetokodi

Copy link
Copy Markdown
Contributor Author

I'd want to fix the riscv issues before this is merged;

I also noticed (in target.cmake) that for the tizen target x64, x86, arm, aarch64, and riscv64 are all supported, should I add riscv32 there too?

@ksh8281

ksh8281 commented Dec 10, 2025

Copy link
Copy Markdown
Collaborator

Sounds good!

Fix 64 bit compares on riscv32

Signed-off-by: Máté Tokodi mate.tokodi@szteszoftver.hu
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