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.

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

Make sure that the sancus examples can be compiled with the SLLVM-based toolchain #7

Open
wants to merge 42 commits into
base: master
Choose a base branch
from

Conversation

hanswinderix
Copy link

No description provided.

Hans Winderix and others added 30 commits December 28, 2018 13:02
@jovanbulck
Copy link
Member

@hanswinderix I realize this never got merged and is probably horribly outdated by now? I'd say it looks like the existing sancus-compiler and sllvm will continue to co-exist at least for the foreseeable time.

Do you plan to continue supporting the sancus-examples compilation with sllvm? If so, I think it'd make sense to add an entry to the Github Actions Workflow matrix so every example is automatically built and ran with both sancus-compiler and sllvm? Do you think this works with your current version of sllvm, or is this simply not something you'd want to support at the moment?

@hanswinderix
Copy link
Author

I have been syncing the SLLVM forks of sancus-examples and sancus-support regulary for more than one year with the upstream versions, but eventually stopped doing this because of the overhead involved and the plans to port Sancus from MSP430 to RISC-V. I have been told that a master thesis student is going to work on this port. This would be a good opportunity to base the RISC-V Sancus compiler on SLLVM instead of the legacy toolchain.

@jovanbulck
Copy link
Member

makes sense. thanks for clarifying!

Let's simply leave this PR open for now then.

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.

2 participants