Add pragma to ensure neopixel_write is always -Os

tannewt · web-flow · commit a3a690dc018b · 2018-11-13T10:56:00.000-08:00
diff --git a/ports/atmel-samd/common-hal/neopixel_write/__init__.c b/ports/atmel-samd/common-hal/neopixel_write/__init__.c
@@ -48,6 +48,11 @@
     }
 #endif
 
+// Ensure this code is compiled with -Os. Any other optimization level may change the timing of it
+// and break neopixels.
+#pragma GCC push_options
+#pragma GCC optimize ("Os")
+
 uint64_t next_start_tick_ms = 0;
 uint32_t next_start_tick_us = 1000;
 
@@ -183,3 +188,5 @@ void common_hal_neopixel_write(const digitalio_digitalinout_obj_t* digitalinout,
     mp_hal_enable_all_interrupts();
 
 }
+
+#pragma GCC pop_options

Original file line number	Diff line number	Diff line change
`@@ -48,6 +48,11 @@`
`48`	`48`	`}`
`49`	`49`	`#endif`
`50`	`50`
	`51`	`+// Ensure this code is compiled with -Os. Any other optimization level may change the timing of it`
	`52`	`+// and break neopixels.`
	`53`	`+#pragma GCC push_options`
	`54`	`+#pragma GCC optimize ("Os")`
	`55`	`+`
`51`	`56`	`uint64_t next_start_tick_ms = 0;`
`52`	`57`	`uint32_t next_start_tick_us = 1000;`
`53`	`58`
`@@ -183,3 +188,5 @@ void common_hal_neopixel_write(const digitalio_digitalinout_obj_t* digitalinout,`
`183`	`188`	`mp_hal_enable_all_interrupts();`
`184`	`189`
`185`	`190`	`}`
	`191`	`+`
	`192`	`+#pragma GCC pop_options`