Skip to content

Conversation

@mcourteaux
Copy link
Contributor

I put this in a few years ago.

@mcourteaux
Copy link
Contributor Author

mcourteaux commented Jan 27, 2025

I'm very tempted to not waste 10 CPU hours for this PR and merge it without running the buildbots 😂 The issue was in there for years, and was never caught... I'll let someone else decide.

@abadams
Copy link
Member

abadams commented Jan 27, 2025

Better to just let the buildbots do it, even when the PR looks impossible-to-be-wrong. It can catch obscure compilation failures on some platforms, or existing build failures we didn't know about yet.

@mcourteaux mcourteaux merged commit 53afee7 into halide:main Jan 27, 2025
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants