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

Cesium leaves a lot of useless conv.u instructions everywhere #515

Open
ForNeVeR opened this issue Jan 16, 2024 · 0 comments
Open

Cesium leaves a lot of useless conv.u instructions everywhere #515

ForNeVeR opened this issue Jan 16, 2024 · 0 comments
Labels
area:compiler Related to code compilation or type checking kind:feature New feature or request status:help-wanted Open for contributors

Comments

@ForNeVeR
Copy link
Owner

Take almost any code that works with arrays, for example, CodeGenArrayTests.ArrayInitialization, and you'll notice that Cesium leaves a useless conv.u instruction often. And none of those are actually needed, or at least I think they aren't.

I believe that Cesium should get a better stack model, and to only emit conversion instructions such as conv.i or conv.u when really required.

@ForNeVeR ForNeVeR added kind:feature New feature or request status:help-wanted Open for contributors area:compiler Related to code compilation or type checking labels Jan 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:compiler Related to code compilation or type checking kind:feature New feature or request status:help-wanted Open for contributors
Projects
None yet
Development

No branches or pull requests

1 participant