Fcalva/cyclone
1
0
Fork 0
mirror of https://github.com/justinethier/cyclone.git synced 2025-05-23 20:15:05 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

Clarify TODO for unused var warnings

Browse source
This commit is contained in:
Justin Ethier 2021-03-15 23:00:27 -04:00
parent 5fa0fab2d6
commit 8f517d4275
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
docs/C-Calling-Conventions.md
View file

@ -167,6 +167,6 @@ TODO: Are there any complications in referencing vars from `args` rather than di
## TODO
- Track down remaining warnings and fix them
- There are 'unused variable' warnings for variables that are in analysis DB but in reality are optimized out in subsequent phases prior to C generation. It may be necessary to add a special pass over the closure code to determine if variables are really used, and store the results in a new hashtable/cache.
- Limits - will need to enforce a limit at some point to prevent segfaults due to actually running out of memory due to passing to many parameters. Will need to figure out what that limit might be and how to enforce that. Perhaps at 15,000 or more args????
- Need to either remove `inline_function_type` or have a special compilation mode/flag so that we do not use it for compilers that are too strict to use it.