Conversation
The xcode project still shells out to build the generators, which is not ideal. But there's a host-arch test file now that's built with cmake, so we can at least make sure the generators don't rot. Co-authored-by: Claude Code <noreply@anthropic.com>
The HelloiOS build must run on macOS and be invoked twice. It isn't a superbuild, and we wouldn't want to inject a superbuild here anyway.
|
We've noticed a problem here: it seem the Xcode dependency analysis is missing a link somewhere and it requires two builds for the generator output to update after making a change. |
…alide/Halide into abadams/revive_hello_ios-xcode
alexreinking
left a comment
There was a problem hiding this comment.
Approving with the caveat that this isn't perfect, see #9049
|
Haven't merged because this still isn't tested on CI. I don't see how to integrate this into the cmake apps testing setup and it's not really testable via the makefile (because it relies on a cmake install package of Halide). I could have "make distrib" produce a cmake-compatible install package, but that's pretty perverse. I also don't want to make a bespoke runner just for this one app. |
I added a workflow to test the app here. |
|
fuzz_bounds failures: and |
This resurrects the HelloiOS app. Alex and I would really appreciate someone with xcode and an iphone or ipad device taking it for a spin on a real device, because we had a hell of a time just getting it right for the two of us. Start by building and installing (to a local folder) Halide via cmake and then check the README in the app.
Still to do before merging is making CI build the app for simulator on mac.
Fixes #8730