Fix and test HelloiOS app

[Infinoid]

Based on gitter discussion, the HelloiOS app seems to have developed some bit-rot.

Here are the issues I see:

• No README.md
• Compilation of target objects couldn't find HalideRuntime.h, HalideRuntimeMetal.h, HalideBuffer.h
• App is not CI-tested. We won't have the target libraries to link the actual iOS application executable, but we can build and run the generator at least
• No build scripts apart from the xcode project file (a shell script, Makefile or CMakeLists.txt would be helpful for testing)
• .gpu_tile() now requires caller to specify inner Vars/RVars
• Unhelpful error messages like Can't split RVar r37$x into Var v2 due to unnamed Vars & RVars

I think that at a minimum, this fix is needed:

@@ -129,17 +129,18 @@ public:
                     .unroll(c)
                     .gpu_tile(y, yi, 8);
             }
+            RVar rxi{"rxi"}, ryi{"ryi"};
             new_state.update(4)
                 .reorder(c, clobber.x)
                 .unroll(c)
-                .gpu_tile(clobber.x, clobber.y, 1, 1);
+                .gpu_tile(clobber.x, clobber.y, rxi, ryi, 1, 1);
 
             state.dim(0).set_stride(3);
             state.dim(2).set_stride(1).set_extent(3);

I don't have xcode or an iOS device, so I can't test any further than building/generating the Halide pipelines. But after applying the above fix, apparently this failure occurs at runtime:

Assertion failed: (i >= 0 && i < this->dimensions()), function dim, file HalideBuffer.h, line 496.

[Infinoid]

Here's how I tested the parts I could. It's based on a shell script embedded within the xcode project file.

#!/bin/bash

set -ex

c++ reaction_diffusion_2_generator.cpp ../../../tools/GenGen.cpp \
	-std=c++17 \
	-fno-rtti \
	-I ../../../build/include \
	-I ../../../tools \
	-L ../../../build/lib \
	-L ../../../build/src \
	-lHalide \
	-Wl,-rpath,../../../build/lib \
	-Wl,-rpath,../../../build/src \
	-o ./reaction_diffusion_2_generator


echo assuming iOS target
TARGET=arm-64-ios
echo TARGET=$TARGET

#non-GPU
for GEN in init update render; do
	./reaction_diffusion_2_generator -g reaction_diffusion_2_${GEN} -f reaction_diffusion_2_${GEN} -n reaction_diffusion_2_${GEN} -o . target=${TARGET}-user_context
done

# Metal
for GEN in init update render; do
	./reaction_diffusion_2_generator -g reaction_diffusion_2_${GEN} -f reaction_diffusion_2_metal_${GEN} -n reaction_diffusion_2_metal_${GEN} -o . target=${TARGET}-metal-user_context
done

[abadams]

Note

Automated triage: this issue was flagged by Claude as likely safe to close, for the reason below. The reasoning was checked against the current source, and a second pass tried and failed to refute it, but no human verified this specific text. If the reason is wrong, please reopen the issue and say so — some fraction of these are expected to be wrong.

Reason (CLOSE_FIXED): The app was revived: it now has a README, a CMake-driven generator build plus setup.sh, an Xcode workspace, gpu_tile calls updated to pass inner Vars, and macOS CI builds it.

Evidence: Commit bcdaedbb4 "Revive the HelloiOS app (#9013)"; apps/HelloiOS/README.md, apps/HelloiOS/setup.sh, apps/HelloiOS/Generators/CMakeLists.txt; apps/HelloiOS/Generators/reaction_diffusion_2_generator.cpp:19,103,175 all use the gpu_tile(x, y, xi, yi, ...) form; CI at .github/workflows/testing-macos.yml:149-164 configures and builds HelloiOS via xcodebuild.

Fixed by #9013 (bcdaedbb4), which revived the app: it now has a README.md, a CMake generator build plus setup.sh, updated gpu_tile calls with explicit inner Vars, and macOS CI coverage in .github/workflows/testing-macos.yml. Closing.