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

literal constant optimization bug #92

Open
HPCguy opened this issue Mar 27, 2023 · 0 comments
Open

literal constant optimization bug #92

HPCguy opened this issue Mar 27, 2023 · 0 comments

Comments

@HPCguy
Copy link
Owner

HPCguy commented Mar 27, 2023

The function Squint.c :: rename_register1() does literal constant optimizations, and is currently unsafe. I suggest commenting out that optimization until this ticket is closed.

This bug might only occur for chained literal constant assignment:

a = b = c = 0.0;

In the case of chained literal assignment, a workaround is likely as easy as:

a = 0.0;
b = 0.0;
c = 0.0;

This bug gives me an excuse to refactor how I handle literal constants in general (which are tricky on ARM since they are embedded in inactive parts of the instruction stream). Sorry that I am not doing a more direct fix, but I do not yet have any heavy users of Squint.c .

Since I am currently in the midst of a serious medical issue, I may not have the energy to get to this for at least several more weeks.

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

No branches or pull requests

1 participant