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

Anderson2021 autoscheduler should ignore multitarget #7539

Open
steven-johnson opened this issue Apr 29, 2023 · 3 comments
Open

Anderson2021 autoscheduler should ignore multitarget #7539

steven-johnson opened this issue Apr 29, 2023 · 3 comments

Comments

@steven-johnson
Copy link
Contributor

If you are building with multitarget and specify the Anderson2021 autoscheduler, the schedule file produced will contain multiple identical copies of the same schedule; this is a waste of time (since we compute the same schedule multiple times) and confusion (since the schedule is the same). We should improve this, but it requires some additional plumbing to do it cleanly.

steven-johnson added a commit that referenced this issue Apr 29, 2023
Partial fix for now
steven-johnson added a commit that referenced this issue May 2, 2023
* Workaround for Issue #7539

Partial fix for now

* trigger buildbots
@antonysigma
Copy link
Contributor

antonysigma commented Jul 20, 2023

Hi @steven-johnson , the ASAN reports a segfault when I use Anderson2021 autoscheduler with multi-output generators. Is multi-output equivalent to multi-target in Halide?

Is the segfault (link below) a feature or a bug? The stack trace contains a function compile_multitarget(), but I am not sure if it is related.
comp-imaging/ProxImaL#75

@steven-johnson
Copy link
Contributor Author

This might be a duplicate of #7606 -- can you offer a repro case of what you are seeing?

Is multi-output equivalent to multi-target in Halide?

Yes.

@antonysigma
Copy link
Contributor

This might be a duplicate of #7606 -- can you offer a repro case of what you are seeing?

Sure. I will submit a draft PR featuring the repro case. Moving the discussion to #7606 .

ardier pushed a commit to ardier/Halide-mutation that referenced this issue Mar 3, 2024
* Workaround for Issue halide#7539

Partial fix for now

* trigger buildbots
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

No branches or pull requests

2 participants