search

AgonConsole8 / agon-vdp

Official Firmware for the Agon Console8: ESP32 VDP
MIT License
38 stars 17 forks source link

This seems to fix the crash but I have no idea why! #166

Closed movievertigo closed 6 months ago

stevesims commented 6 months ago

as per comments on discord...

my investigations seem to show that this is a compiler optimisation flags issue. something about building with -O3 seems to be meaning that the >=0 test becomes slightly unreliable. if the build flag is changed to -O2 then the test appears to become reliable. there also seems to be no discernible performance penalty for using -O2 over -O3

for a "belt and braces" approach, we should probably both swap to -O2 and change these equality tests

movievertigo commented 6 months ago

Closed as it's no longer relevant