Gate MSL infinite loop optimization workaround to a flag enabled by default

naga/src/back/msl/writer.rs

@@ -3028,7 +3028,10 @@ impl<W: Write> Writer<W> {
                     ref continuing,
                     break_if,
                 } => {
-                    self.emit_loop_reachable_macro()?;
+                    // We only emit the macro if the index policy is not checked.
+                    if context.expression.policies.index != index::BoundsCheckPolicy::Unchecked {
+                        self.emit_loop_reachable_macro()?;
+                    }
                     if !continuing.is_empty() || break_if.is_some() {
                         let gate_name = self.namer.call("loop_init");
                         writeln!(self.out, "{level}bool {gate_name} = true;")?;

naga/src/proc/index.rs

@@ -67,6 +67,7 @@ pub enum BoundsCheckPolicy {
 pub struct BoundsCheckPolicies {
     /// How should the generated code handle array, vector, or matrix indices
     /// that are out of range?
+    /// On Metal, this policy also dictates how loops are checked for UB.
     #[cfg_attr(feature = "deserialize", serde(default))]
     pub index: BoundsCheckPolicy,