Optimize WS2812FX::service()

TripleWhy · TripleWhy · commit 91a7e7dbbcc4 · 2025-02-12T19:03:27.000+01:00
diff --git a/wled00/FX_fcn.cpp b/wled00/FX_fcn.cpp
@@ -1404,6 +1404,8 @@ void WS2812FX::finalizeInit() {
   deserializeMap();     // (re)load default ledmap (will also setUpMatrix() if ledmap does not exist)
 }
 
+#pragma GCC push_options
+#pragma GCC optimize ("O3")
 void WS2812FX::service() {
   unsigned long nowUp = millis(); // Be aware, millis() rolls over every 49 days
   now = nowUp + timebase;
@@ -1568,6 +1570,7 @@ void WS2812FX::service() {
   if ((_targetFps != FPS_UNLIMITED) && (millis() - nowUp > _frametime)) DEBUG_PRINTF_P(PSTR("Slow strip %u/%d.\n"), (unsigned)(millis()-nowUp), (int)_frametime);
   #endif
 }
+#pragma GCC pop_options
 
 void IRAM_ATTR WS2812FX::setPixelColor(unsigned i, uint32_t col) const {
   i = getMappedPixelIndex(i);

Original file line number	Diff line number	Diff line change
`@@ -1404,6 +1404,8 @@ void WS2812FX::finalizeInit() {`
`1404`	`1404`	`deserializeMap(); // (re)load default ledmap (will also setUpMatrix() if ledmap does not exist)`
`1405`	`1405`	`}`
`1406`	`1406`
	`1407`	`+#pragma GCC push_options`
	`1408`	`+#pragma GCC optimize ("O3")`
`1407`	`1409`	`void WS2812FX::service() {`
`1408`	`1410`	`unsigned long nowUp = millis(); // Be aware, millis() rolls over every 49 days`
`1409`	`1411`	`now = nowUp + timebase;`
`@@ -1568,6 +1570,7 @@ void WS2812FX::service() {`
`1568`	`1570`	`if ((_targetFps != FPS_UNLIMITED) && (millis() - nowUp > _frametime)) DEBUG_PRINTF_P(PSTR("Slow strip %u/%d.\n"), (unsigned)(millis()-nowUp), (int)_frametime);`
`1569`	`1571`	`#endif`
`1570`	`1572`	`}`
	`1573`	`+#pragma GCC pop_options`
`1571`	`1574`
`1572`	`1575`	`void IRAM_ATTR WS2812FX::setPixelColor(unsigned i, uint32_t col) const {`
`1573`	`1576`	`i = getMappedPixelIndex(i);`