Simplify logic to work around hang.

Author: maleadt

src/compiler/execution.jl

@@ -277,12 +277,13 @@ end
         throw(ArgumentError("Number of threads in group ($(threads.width * threads.height * threads.depth)) should not exceed $(kernel.pipeline.maxTotalThreadsPerThreadgroup)"))
 
     cmdbuf = if kernel.loggingEnabled
+        # TODO: make this a dynamic error, i.e., from the kernel (JuliaGPU/Metal.jl#433)
         if macos_version() < v"15"
-            @error "Logging is only supported on macOS 15 or higher"
+            error("Logging is only supported on macOS 15 or higher")
         end
 
         if MTLCaptureManager().isCapturing
-            @error "Logging is not supported while GPU frame capturing"
+            error("Logging is not supported while GPU frame capturing")
         end
 
         log_state_descriptor = MTLLogStateDescriptor()

src/device/intrinsics/output.jl

@@ -48,11 +48,7 @@ macro mtlprintf(fmt::String, args...)
     fmt_val = Val(Symbol(fmt))
 
     quote
-        if metal_version() < sv"3.2"
-            @print_and_throw "@mtlprintf requires Metal 3.2 (macOS 15) or higher"
-        else
-            _mtlprintf($fmt_val, $(map(arg -> :(promote_c_argument($arg)), esc.(args))...))
-        end
+        _mtlprintf($fmt_val, $(map(arg -> :(promote_c_argument($arg)), esc.(args))...))
     end
 end
 

test/output.jl

@@ -4,6 +4,12 @@ if Metal.macos_version() < v"15"
 
 @warn "Skipping output tests in macOS 14 and below"
 
+function kernel()
+    @mtlprint("Hello, World\n")
+    return
+end
+@test_throws "Logging is only supported on macOS 15 or higher" @metal kernel()
+
 else
 
 @testset "formatted output" begin