Skip to content

Fix CUDA HTML code printing bug.#8558

Merged
mcourteaux merged 2 commits intohalide:mainfrom
mcourteaux:fix-html-codegen-cuda
Jan 27, 2025
Merged

Fix CUDA HTML code printing bug.#8558
mcourteaux merged 2 commits intohalide:mainfrom
mcourteaux:fix-html-codegen-cuda

Conversation

@mcourteaux
Copy link
Copy Markdown
Contributor

I put this in a few years ago.

Comment thread src/StmtToHTML.cpp Outdated
@mcourteaux
Copy link
Copy Markdown
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
Copy Markdown
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
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