Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix corner case in if_then_else simplification #8189

Merged
merged 2 commits into from
Apr 16, 2024
Merged

Conversation

abadams
Copy link
Member

@abadams abadams commented Apr 11, 2024

Fixes the segfault reported in #8170

I have no idea how to repro this in a test. It came about as a result of applying the adams autoscheduler to an hvx pipeline, which produced a schedule with vectors wider than tiles, which turned into predication, which was then turned into something invalid inside the hexagon lowering path.

The if_then_else intrinsic is supposed to evaluate to zero if the condition is false and there's no false value defined.

@steven-johnson
Copy link
Contributor

Ready to land?

@abadams abadams merged commit 7994e70 into main Apr 16, 2024
19 checks passed
@jxl1080
Copy link

jxl1080 commented May 8, 2024

Hi, I wonder if this fix will eventually be included in binaries that I can download from: https://github.com/halide/Halide/releases

If so, is there an estimation when it will become publicly available?

Thank you!

@abadams
Copy link
Member Author

abadams commented May 9, 2024

There are one or two bugs that I would like to close out before we next do an official release, so it'll be a few more weeks. In the mean-time you can download a nightly build from https://buildbot.halide-lang.org/

These are the same as the releases. For the release artifacts we just grab the latest nightlies.

@jxl1080
Copy link

jxl1080 commented May 9, 2024

@abadams Hi Andrew, thank you for your quick response! However, I'm still not very clear which item on https://buildbot.halide-lang.org/ is the nightly build you were referring to. I tried 'Halide-17.0.1-x86-64-linux-52541176253e74467dabc42eeee63d9a62c199f6.tar' on that page, but the issue I reported in #8170 seems still over there. Am I downloading the wrong item on that page? Thank you.

@jxl1080
Copy link

jxl1080 commented May 9, 2024

I got more experimental result.
The issue is resolved in 'Halide-18.0.0-x86-64-linux-dfaf6ad0715bc4973955d085c40f13340971529b', but it's still over there for 'Halide-17.0.0-x86-64-linux-d15325e3df2fec9927184bdc5a06284477df84bf' and 'Halide-17.0.1-x86-64-linux-52541176253e74467dabc42eeee63d9a62c199f6'.
Will the fix eventually be in version 17, or I have to use version 18 if I need the fix? Thank you.

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