Halide icon indicating copy to clipboard operation
Halide copied to clipboard

Fix autoscheduler failure on image loads with single point bounds for arguments. Add tests.

Open zvookin opened this issue 4 years ago • 4 comments

This is for Andrew to look at per previous discussion.

zvookin avatar Apr 20 '21 06:04 zvookin

What should I run to repro? It seems to pass for me.

abadams avatar Apr 20 '21 15:04 abadams

@abadams is going to take this over as there are a few subtleties involved. It may be a week or two due to other commitments.

zvookin avatar Apr 28 '21 19:04 zvookin

@zvookin Finally got around to this. It did indeed turn out to be subtle. scalar image loads were sneaking in via various scopes (e.g. the rdom bounds can use them), so removing them had to be done as the final step. PTAL

abadams avatar Jun 18 '21 19:06 abadams

Is this PR still active? Should it be closed?

steven-johnson avatar Nov 15 '21 19:11 steven-johnson