Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

gcc: add new rv32emc configurations #744

Merged
merged 1 commit into from
Apr 8, 2024
Merged

gcc: add new rv32emc configurations #744

merged 1 commit into from
Apr 8, 2024

Conversation

masz-nordic
Copy link
Contributor

Namely:

  • rv32emc_zicsr
  • rv32emc_zicsr_zba_zbb_zbc_zbs
  • rv32emc_zicsr_zicntr_zba_zbb_zbc_zbs
  • rv32emc_zicsr_zicntr_zba_zbb_zbc_zbs_zcb

@masz-nordic masz-nordic mentioned this pull request Apr 5, 2024
Merged
@carlescufi
Copy link
Member

@stephanosio @masz-nordic
I see CI is failing with:

Error:     checking for suffix of object files... configure: error: in `/__w/_temp/workspace/build/.build/riscv64-zephyr-elf/build/build-cc-gcc-core/riscv64-zephyr-elf/rv32emc_zicsr_zicntr_zba_zbb_zbc_zbs/ilp32e/libgcc':
Error: [ERROR]    configure: error: cannot compute suffix of object files: cannot compile
Error:     make[1]: *** [configure-target-libgcc] Error 1

@masz-nordic
Copy link
Contributor Author

Yup, waiting for CI to finish so I can take a look at logs.

@masz-nordic masz-nordic force-pushed the riscv_rv32 branch 2 times, most recently from 92b5239 to c8660c6 Compare April 5, 2024 14:17
@masz-nordic
Copy link
Contributor Author

masz-nordic commented Apr 5, 2024 •
edited
Loading

Removing zicntr and zcb as they are not yet supported in GCC version used in SDK.

@masz-nordic masz-nordic force-pushed the riscv_rv32 branch 2 times, most recently from 5e3b867 to 0f7129c Compare April 5, 2024 15:15
@stephanosio
Copy link
Member

zephyrproject-rtos/gcc#29 has been merged. Please update the submodule ref to point to the merged commit.

@masz-nordic
Copy link
Contributor Author

@stephanosio done

@stephanosio stephanosio merged commit 6ba6445 into zephyrproject-rtos:main Apr 8, 2024
30 checks passed
@stephanosio stephanosio mentioned this pull request Apr 22, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stephanosio stephanosio stephanosio approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@masz-nordic @carlescufi @stephanosio