halide / Halide

a language for fast, portable data-parallel computation
https://halide-lang.org
Other
5.91k stars 1.07k forks source link

Don't let users disguise RVars as Vars #8441

Closed abadams closed 1 month ago

abadams commented 1 month ago

Fixes #7827

abadams commented 1 month ago

This uncovered an error where our generated schedules from adams2019 were incorrectly declaring some RVars as Vars. The scheduling directives used are the ones that are legal for RVars, so there should not be any incorrect output, but adding this check may cause existing checked-in generated schedules to throw a compile-time error. This can be manually fixed up by changing the declaration to be an RVar.