⚡️ Use fast types where it helps

Marlin/src/core/macros.h

@@ -375,8 +375,8 @@
 #define _JOIN_1(O)         (O)
 #define JOIN_N(N,C,V...)   (DO(JOIN,C,LIST_N(N,V)))
 
-#define LOOP_S_LE_N(VAR, S, N) for (uint8_t VAR=(S); VAR<=(N); VAR++)
-#define LOOP_S_L_N(VAR, S, N) for (uint8_t VAR=(S); VAR<(N); VAR++)
+#define LOOP_S_LE_N(VAR, S, N) for (uint_fast8_t VAR=(S); VAR<=uint_fast8_t(N); VAR++)
+#define LOOP_S_L_N(VAR, S, N) for (uint_fast8_t VAR=(S); VAR<uint_fast8_t(N); VAR++)
 #define LOOP_LE_N(VAR, N) LOOP_S_LE_N(VAR, 0, N)
 #define LOOP_L_N(VAR, N) LOOP_S_L_N(VAR, 0, N)
 

Marlin/src/core/types.h

@@ -161,7 +161,7 @@ typedef struct AxisFlags {
 //  - A_AXIS, B_AXIS, and C_AXIS should be used for Steppers, corresponding to XYZ on Cartesians
 //  - X_HEAD, Y_HEAD, and Z_HEAD should be used for Steppers on Core kinematics
 //
-enum AxisEnum : uint8_t {
+enum AxisEnum : uint_fast8_t {
 
   // Linear axes may be controlled directly or indirectly
   NUM_AXIS_LIST(X_AXIS, Y_AXIS, Z_AXIS, I_AXIS, J_AXIS, K_AXIS, U_AXIS, V_AXIS, W_AXIS)

Marlin/src/feature/bedlevel/bedlevel.cpp

@@ -135,7 +135,7 @@ void reset_bed_level() {
   /**
    * Print calibration results for plotting or manual frame adjustment.
    */
-  void print_2d_array(const uint8_t sx, const uint8_t sy, const uint8_t precision, const float *values) {
+  void print_2d_array(const uint_fast8_t sx, const uint_fast8_t sy, const uint_fast8_t precision, const float *values) {
     #ifndef SCAD_MESH_OUTPUT
       LOOP_L_N(x, sx) {
         serial_spaces(precision + (x < 10 ? 3 : 2));
@@ -162,7 +162,7 @@ void reset_bed_level() {
         }
         else {
           #ifdef SCAD_MESH_OUTPUT
-            for (uint8_t i = 3; i < precision + 3; i++)
+            for (uint_fast8_t i = 3; i < precision + 3; i++)
               SERIAL_CHAR(' ');
             SERIAL_ECHOPGM("NAN");
           #else

Marlin/src/feature/bedlevel/bedlevel.h

@@ -73,12 +73,12 @@ class TemporaryBedLevelingState {
 
     #include <stdint.h>
 
-    typedef float (*element_2d_fn)(const uint8_t, const uint8_t);
+    typedef float (*element_2d_fn)(const uint_fast8_t, const uint_fast8_t);
 
     /**
      * Print calibration results for plotting or manual frame adjustment.
      */
-    void print_2d_array(const uint8_t sx, const uint8_t sy, const uint8_t precision, const float *values);
+    void print_2d_array(const uint_fast8_t sx, const uint_fast8_t sy, const uint_fast8_t precision, const float *values);
 
   #endif
 

Marlin/src/feature/mmu/mmu2.cpp

@@ -477,7 +477,7 @@ static void mmu2_not_responding() {
   /**
    * Handle tool change
    */
-  void MMU2::tool_change(const uint8_t index) {
+  void MMU2::tool_change(const uint_fast8_t index) {
 
     if (!_enabled) return;
 
@@ -559,7 +559,7 @@ static void mmu2_not_responding() {
   /**
    * Handle tool change
    */
-  void MMU2::tool_change(const uint8_t index) {
+  void MMU2::tool_change(const uint_fast8_t index) {
     if (!_enabled) return;
 
     set_runout_valid(false);
@@ -684,7 +684,7 @@ static void mmu2_not_responding() {
   /**
    * Handle tool change
    */
-  void MMU2::tool_change(const uint8_t index) {
+  void MMU2::tool_change(const uint_fast8_t index) {
     if (!_enabled) return;
 
     set_runout_valid(false);

Marlin/src/feature/mmu/mmu2.h

@@ -45,7 +45,7 @@ class MMU2 {
   static void reset();
   static bool enabled() { return _enabled; }
   static void mmu_loop();
-  static void tool_change(const uint8_t index);
+  static void tool_change(const uint_fast8_t index);
   static void tool_change(const char *special);
   static uint8_t get_current_tool();
   static void set_filament_type(const uint8_t index, const uint8_t type);

Marlin/src/feature/pause.cpp

@@ -227,7 +227,7 @@ bool load_filament(const_float_t slow_load_length/*=0*/, const_float_t fast_load
   if (show_lcd) ui.pause_show_message(PAUSE_MESSAGE_LOAD, mode);
 
   #if ENABLED(DUAL_X_CARRIAGE)
-    const int8_t saved_ext        = active_extruder;
+    const int_fast8_t saved_ext   = active_extruder;
     const bool saved_ext_dup_mode = extruder_duplication_enabled;
     set_duplication_enabled(false, DXC_ext);
   #endif
@@ -468,7 +468,7 @@ bool pause_print(const_float_t retract, const xyz_pos_t &park_point, const bool
   TERN_(DWIN_LCD_PROUI, if (!do_park) ui.set_status(GET_TEXT_F(MSG_PARK_FAILED)));
 
   #if ENABLED(DUAL_X_CARRIAGE)
-    const int8_t saved_ext        = active_extruder;
+    const int_fast8_t saved_ext   = active_extruder;
     const bool saved_ext_dup_mode = extruder_duplication_enabled;
     set_duplication_enabled(false, DXC_ext);
   #endif
@@ -523,7 +523,7 @@ void wait_for_confirmation(const bool is_reload/*=false*/, const int8_t max_beep
   HOTEND_LOOP() thermalManager.heater_idle[e].start(nozzle_timeout);
 
   #if ENABLED(DUAL_X_CARRIAGE)
-    const int8_t saved_ext        = active_extruder;
+    const int_fast8_t saved_ext   = active_extruder;
     const bool saved_ext_dup_mode = extruder_duplication_enabled;
     set_duplication_enabled(false, DXC_ext);
   #endif

Marlin/src/feature/powerloss.h

@@ -71,7 +71,7 @@ typedef struct {
     xyz_pos_t position_shift;
   #endif
   #if HAS_MULTI_EXTRUDER
-    uint8_t active_extruder;
+    uint_fast8_t active_extruder;
   #endif
 
   #if DISABLED(NO_VOLUMETRICS)
@@ -85,7 +85,7 @@ typedef struct {
     celsius_t target_temperature_bed;
   #endif
   #if HAS_FAN
-    uint8_t fan_speed[FAN_COUNT];
+    uint_fast8_t fan_speed[FAN_COUNT];
   #endif
 
   #if HAS_LEVELING

Marlin/src/feature/runout.h

@@ -145,11 +145,11 @@ class TFilamentMonitor : public FilamentMonitorBase {
             }
           #else
             const bool ran_out = TEST(runout_flags, active_extruder);  // suppress non active extruders
-            uint8_t extruder = active_extruder;
+            uint_fast8_t extruder = active_extruder;
           #endif
         #else
           const bool ran_out = !!runout_flags;
-          uint8_t extruder = active_extruder;
+          uint_fast8_t extruder = active_extruder;
         #endif
 
         #if ENABLED(FILAMENT_RUNOUT_SENSOR_DEBUG)

Marlin/src/gcode/calibrate/G28.cpp

@@ -341,7 +341,7 @@ void GcodeSuite::G28() {
   // Always home with tool 0 active
   #if HAS_MULTI_HOTEND
     #if DISABLED(DELTA) || ENABLED(DELTA_HOME_TO_SAFE_ZONE)
-      const uint8_t old_tool_index = active_extruder;
+      const uint_fast8_t old_tool_index = active_extruder;
     #endif
     // PARKING_EXTRUDER homing requires different handling of movement / solenoid activation, depending on the side of homing
     #if ENABLED(PARKING_EXTRUDER)

Marlin/src/gcode/config/M200-M205.cpp

@@ -41,7 +41,7 @@
     if (!parser.seen("DST" TERN_(VOLUMETRIC_EXTRUDER_LIMIT, "L")))
       return M200_report();
 
-    const int8_t target_extruder = get_target_extruder_from_command();
+    const int_fast8_t target_extruder = get_target_extruder_from_command();
     if (target_extruder < 0) return;
 
     bool vol_enable = parser.volumetric_enabled,
@@ -125,7 +125,7 @@ void GcodeSuite::M201() {
   if (!parser.seen("T" STR_AXES_LOGICAL TERN_(XY_FREQUENCY_LIMIT, "FS")))
     return M201_report();
 
-  const int8_t target_extruder = get_target_extruder_from_command();
+  const int_fast8_t target_extruder = get_target_extruder_from_command();
   if (target_extruder < 0) return;
 
   #ifdef XY_FREQUENCY_LIMIT
@@ -179,7 +179,7 @@ void GcodeSuite::M203() {
   if (!parser.seen("T" STR_AXES_LOGICAL))
     return M203_report();
 
-  const int8_t target_extruder = get_target_extruder_from_command();
+  const int_fast8_t target_extruder = get_target_extruder_from_command();
   if (target_extruder < 0) return;
 
   LOOP_LOGICAL_AXES(i)

Marlin/src/gcode/config/M217.cpp

@@ -140,7 +140,7 @@ void GcodeSuite::M217() {
     if (parser.seen('T')) {     // Migrate now
       if (parser.has_value()) {
         const int16_t tval = parser.value_int();
-        if (WITHIN(tval, 0, EXTRUDERS - 1) && tval != active_extruder) {
+        if (WITHIN(tval, 0, EXTRUDERS - 1) && tval != signed(active_extruder)) {
           migration.target = tval + 1;
           extruder_migration();
           migration.target = 0; // disable

Marlin/src/gcode/config/M218.cpp

@@ -43,7 +43,7 @@ void GcodeSuite::M218() {
 
   if (!parser.seen_any()) return M218_report();
 
-  const int8_t target_extruder = get_target_extruder_from_command();
+  const int_fast8_t target_extruder = get_target_extruder_from_command();
   if (target_extruder < 0) return;
 
   if (parser.seenval('X')) hotend_offset[target_extruder].x = parser.value_linear_units();

Marlin/src/gcode/config/M221.cpp

@@ -30,7 +30,7 @@
  */
 void GcodeSuite::M221() {
 
-  const int8_t target_extruder = get_target_extruder_from_command();
+  const int_fast8_t target_extruder = get_target_extruder_from_command();
   if (target_extruder < 0) return;
 
   if (parser.seenval('S'))

Marlin/src/gcode/config/M92.cpp

@@ -39,7 +39,7 @@
  */
 void GcodeSuite::M92() {
 
-  const int8_t target_extruder = get_target_extruder_from_command();
+  const int_fast8_t target_extruder = get_target_extruder_from_command();
   if (target_extruder < 0) return;
 
   // No arguments? Show M92 report.

Marlin/src/gcode/control/T.cpp

@@ -46,7 +46,7 @@
  *   Tx   Same as T?, but nozzle doesn't have to be preheated. Tc requires a preheated nozzle to finish filament load.
  *   Tc   Load to nozzle after filament was prepared by Tc and nozzle is already heated.
  */
-void GcodeSuite::T(const int8_t tool_index) {
+void GcodeSuite::T(const int_fast8_t tool_index) {
 
   DEBUG_SECTION(log_T, "T", DEBUGGING(LEVELING));
   if (DEBUGGING(LEVELING)) DEBUG_ECHOLNPGM("...(", tool_index, ")");

Marlin/src/gcode/feature/pause/M600.cpp

@@ -73,7 +73,7 @@
 void GcodeSuite::M600() {
 
   #if ENABLED(MIXING_EXTRUDER)
-    const int8_t eindex = get_target_e_stepper_from_command();
+    const int_fast8_t eindex = get_target_e_stepper_from_command();
     if (eindex < 0) return;
 
     const uint8_t old_mixing_tool = mixer.get_current_vtool();
@@ -82,9 +82,9 @@ void GcodeSuite::M600() {
     MIXER_STEPPER_LOOP(i) mixer.set_collector(i, i == uint8_t(eindex) ? 1.0 : 0.0);
     mixer.normalize();
 
-    const int8_t target_extruder = active_extruder;
+    const int_fast8_t target_extruder = active_extruder;
   #else
-    const int8_t target_extruder = get_target_extruder_from_command();
+    const int_fast8_t target_extruder = get_target_extruder_from_command();
     if (target_extruder < 0) return;
   #endif
 
@@ -112,7 +112,7 @@ void GcodeSuite::M600() {
 
   #if HAS_MULTI_EXTRUDER
     // Change toolhead if specified
-    const uint8_t active_extruder_before_filament_change = active_extruder;
+    const uint_fast8_t active_extruder_before_filament_change = active_extruder;
     if (active_extruder != target_extruder && TERN1(DUAL_X_CARRIAGE, !idex_is_duplicating()))
       tool_change(target_extruder);
   #endif

Marlin/src/gcode/feature/pause/M603.cpp

@@ -44,7 +44,7 @@ void GcodeSuite::M603() {
 
   if (!parser.seen("TUL")) return M603_report();
 
-  const int8_t target_extruder = get_target_extruder_from_command();
+  const int_fast8_t target_extruder = get_target_extruder_from_command();
   if (target_extruder < 0) return;
 
   // Unload length

Marlin/src/gcode/feature/pause/M701_M702.cpp

@@ -60,7 +60,7 @@ void GcodeSuite::M701() {
   if (TERN0(NO_MOTION_BEFORE_HOMING, axes_should_home())) park_point.z = 0;
 
   #if ENABLED(MIXING_EXTRUDER)
-    const int8_t eindex = get_target_e_stepper_from_command();
+    const int_fast8_t eindex = get_target_e_stepper_from_command();
     if (eindex < 0) return;
 
     const uint8_t old_mixing_tool = mixer.get_current_vtool();
@@ -69,9 +69,9 @@ void GcodeSuite::M701() {
     MIXER_STEPPER_LOOP(i) mixer.set_collector(i, i == uint8_t(eindex) ? 1.0 : 0.0);
     mixer.normalize();
 
-    const int8_t target_extruder = active_extruder;
+    const int_fast8_t target_extruder = active_extruder;
   #else
-    const int8_t target_extruder = get_target_extruder_from_command();
+    const int_fast8_t target_extruder = get_target_extruder_from_command();
     if (target_extruder < 0) return;
   #endif
 
@@ -83,7 +83,7 @@ void GcodeSuite::M701() {
 
   #if HAS_MULTI_EXTRUDER && (HAS_PRUSA_MMU1 || !HAS_MMU)
     // Change toolhead if specified
-    uint8_t active_extruder_before_filament_change = active_extruder;
+    uint_fast8_t active_extruder_before_filament_change = active_extruder;
     if (active_extruder != target_extruder)
       tool_change(target_extruder);
   #endif
@@ -165,16 +165,16 @@ void GcodeSuite::M702() {
     #endif
 
     if (seenT) {
-      const int8_t eindex = get_target_e_stepper_from_command();
+      const int_fast8_t eindex = get_target_e_stepper_from_command();
       if (eindex < 0) return;
       mixer.T(MIXER_DIRECT_SET_TOOL);
       MIXER_STEPPER_LOOP(i) mixer.set_collector(i, i == uint8_t(eindex) ? 1.0 : 0.0);
       mixer.normalize();
     }
 
-    const int8_t target_extruder = active_extruder;
+    const int_fast8_t target_extruder = active_extruder;
   #else
-    const int8_t target_extruder = get_target_extruder_from_command();
+    const int_fast8_t target_extruder = get_target_extruder_from_command();
     if (target_extruder < 0) return;
   #endif
 
@@ -186,7 +186,7 @@ void GcodeSuite::M702() {
 
   #if HAS_MULTI_EXTRUDER && (HAS_PRUSA_MMU1 || !HAS_MMU)
     // Change toolhead if specified
-    uint8_t active_extruder_before_filament_change = active_extruder;
+    uint_fast8_t active_extruder_before_filament_change = active_extruder;
     if (active_extruder != target_extruder)
       tool_change(target_extruder);
   #endif

Marlin/src/gcode/feature/trinamic/M906.cpp

@@ -129,7 +129,7 @@ void GcodeSuite::M906() {
 
       #if AXIS_IS_TMC(E0) || AXIS_IS_TMC(E1) || AXIS_IS_TMC(E2) || AXIS_IS_TMC(E3) || AXIS_IS_TMC(E4) || AXIS_IS_TMC(E5) || AXIS_IS_TMC(E6) || AXIS_IS_TMC(E7)
         case E_AXIS: {
-          const int8_t eindex = get_target_e_stepper_from_command(-2);
+          const int_fast8_t eindex = get_target_e_stepper_from_command(-2);
           #if AXIS_IS_TMC(E0)
             if (eindex < 0 || eindex == 0) TMC_SET_CURRENT(E0);
           #endif

Marlin/src/gcode/feature/trinamic/M911-M914.cpp

@@ -335,7 +335,7 @@
 
         #if E0_HAS_STEALTHCHOP || E1_HAS_STEALTHCHOP || E2_HAS_STEALTHCHOP || E3_HAS_STEALTHCHOP || E4_HAS_STEALTHCHOP || E5_HAS_STEALTHCHOP || E6_HAS_STEALTHCHOP || E7_HAS_STEALTHCHOP
           case E_AXIS: {
-            const int8_t eindex = get_target_e_stepper_from_command(-2);
+            const int_fast8_t eindex = get_target_e_stepper_from_command(-2);
             TERN_(E0_HAS_STEALTHCHOP, if (eindex < 0 || eindex == 0) TMC_SET_PWMTHRS_E(0));
             TERN_(E1_HAS_STEALTHCHOP, if (eindex < 0 || eindex == 1) TMC_SET_PWMTHRS_E(1));
             TERN_(E2_HAS_STEALTHCHOP, if (eindex < 0 || eindex == 2) TMC_SET_PWMTHRS_E(2));
@@ -345,7 +345,7 @@
             TERN_(E6_HAS_STEALTHCHOP, if (eindex < 0 || eindex == 6) TMC_SET_PWMTHRS_E(6));
             TERN_(E7_HAS_STEALTHCHOP, if (eindex < 0 || eindex == 7) TMC_SET_PWMTHRS_E(7));
           } break;
-        #endif // E_STEPPERS
+        #endif // E*_HAS_STEALTHCHOP
       }
     }
 

Marlin/src/gcode/gcode.cpp

@@ -134,7 +134,7 @@ void GcodeSuite::say_units() {
  * Get the target extruder from the T parameter or the active_extruder
  * Return -1 if the T parameter is out of range
  */
-int8_t GcodeSuite::get_target_extruder_from_command() {
+int_fast8_t GcodeSuite::get_target_extruder_from_command() {
   if (parser.seenval('T')) {
     const int8_t e = parser.value_byte();
     if (e < EXTRUDERS) return e;
@@ -151,7 +151,7 @@ int8_t GcodeSuite::get_target_extruder_from_command() {
  * If there is no 'T' parameter then dval will be substituted.
  * Returns -1 if the resulting E stepper index is out of range.
  */
-int8_t GcodeSuite::get_target_e_stepper_from_command(const int8_t dval/*=-1*/) {
+int_fast8_t GcodeSuite::get_target_e_stepper_from_command(const int_fast8_t dval/*=-1*/) {
   const int8_t e = parser.intval('T', dval);
   if (WITHIN(e, 0, E_STEPPERS - 1)) return e;
   if (dval == -2) return dval;

Marlin/src/gcode/gcode.h

@@ -420,8 +420,8 @@ class GcodeSuite {
   }
   static void say_units();
 
-  static int8_t get_target_extruder_from_command();
-  static int8_t get_target_e_stepper_from_command(const int8_t dval=-1);
+  static int_fast8_t get_target_extruder_from_command();
+  static int_fast8_t get_target_e_stepper_from_command(const int_fast8_t dval=-1);
   static void get_destination_from_command();
 
   static void process_parsed_command(const bool no_ok=false);
@@ -1254,7 +1254,7 @@ class GcodeSuite {
     static void M710_report(const bool forReplay=true);
   #endif
 
-  static void T(const int8_t tool_index);
+  static void T(const int_fast8_t tool_index);
 
 };