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

Add new platform(s): Foenix F256 Jr & K #263

Open
bcampbell opened this issue Dec 9, 2023 · 5 comments
Open

Add new platform(s): Foenix F256 Jr & K #263

bcampbell opened this issue Dec 9, 2023 · 5 comments
Labels
enhancement New feature or request p2

Comments

@bcampbell
Copy link

https://c256foenix.com/f256k and https://c256foenix.com/f256-jr
I think they're similar enough to just be a single target.

I've got a very rough first pass going at: https://github.com/bcampbell/llvm-mos-sdk/tree/f256k

It's pretty specific to my own current project, but it'd be good to get some critique and feedback and start it heading down the path toward upstream inclusion...

@sduensin
Copy link

I built this and made a simple binary to test with FoenixIDE. Fantastic!

@mysterymath mysterymath added enhancement New feature or request p2 labels Dec 11, 2023
@mysterymath
Copy link
Member

The changes you have look more or less like I'd expect; I think we can hammer out the details in a PR if you're up for it.

@bcampbell
Copy link
Author

I'm going to be traveling overseas for the next month, so I'm probably not going to have a whole lot of time to put into this. Happy for someone else to pick it up, or I'll get stuck into it properly when I have more time for it.

@bcampbell
Copy link
Author

One thing that was a little odd - I had trouble with some for loops which just filled a block of memory with a value. They just weren't working. Turns out they were being optimised to use memset instead. For whatever reason, that seemed to not work right. Using -fno-builtin-memset got the loops behaving as they should.

So I suspect I've screwed up something with the runtime...
TBH I'm pretty shaky on all the merged utility routines to initialise bss, copy data into zp, etc etc... I just added the ones that sounded right and fiddled a bit. So I don't have a lot of confidence I got it 100% right :-)

@sduensin
Copy link

sduensin commented Jan 2, 2024

I had trouble with some for loops which just filled a block of memory with a value. They just weren't working. Turns out they were being optimised to use memset instead. For whatever reason, that seemed to not work right.

Just ran into this myself. -fno-builtin-memset also helped in my case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request p2
Projects
None yet
Development

No branches or pull requests

3 participants