Skip to content
This repository has been archived by the owner on Jan 26, 2022. It is now read-only.

Support more than 85 variables #124

Open
runer112 opened this issue Apr 5, 2019 · 3 comments
Open

Support more than 85 variables #124

runer112 opened this issue Apr 5, 2019 · 3 comments
Labels

Comments

@runer112
Copy link
Collaborator

runer112 commented Apr 5, 2019

After 85 variables, variables no longer get correctly allocated due to running out of index register offset range. At this point, you should fall back to referencing extra variables directly.

@runer112 runer112 added the bug label Apr 5, 2019
@AlvajoyAsante
Copy link

This Limits ICE very much. Even though it cause Allocating error. I forked this and removed the ICE variabel limitation. 😄

@runer112
Copy link
Collaborator Author

runer112 commented Mar 28, 2020

I forked this and removed the ICE variabel limitation. 😄

Don't do that. The limitation is in there for a reason. Accesses to any variables after the 85th will be miscompiled.

EDIT: Probably, anyway.

@AlvajoyAsante
Copy link

Don't do that. The limitation is in there for a reason. Accesses to any variables after the 85th will be miscompiled.

I have not had a problem thus far. I am using ConvHex to limit over sizing.

  • Maybe Convex gets rid of the allocating errors .

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants