SmingHub / Sming

Sming - powerful open source framework simplifying the creation of embedded C++ applications.
https://sming.readthedocs.io
GNU Lesser General Public License v3.0
1.45k stars 349 forks source link

Fix `pgm_read_word` for 64-bit host builds #2844

Closed mikee47 closed 1 week ago

mikee47 commented 1 week ago

Sanitizer caught issue with pgm_read_dword on 64-bit build, defined as unsigned long instead of uint32_t.

Suppress sanitizer alignment warnings for some FlashString methods where (64-bit) pointers are aligned to 32-bit boundaries. We need everything (including pointers) to be 32-bit aligned so this isn't a bug. (It's an optimisation thing.)