search

halide / Halide

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

Fix for top-of-tree LLVM #8442

Closed steven-johnson closed 1 month ago

abadams commented 1 month ago

Does this actually need to be conditional on llvm 20? I think getOrInsertDeclaration has existed for a long time.

steven-johnson commented 1 month ago

Does this actually need to be conditional on llvm 20? I think getOrInsertDeclaration has existed for a long time.

I didn't check.

EDIT: removing conditional.

EDIT 2: restored conditional; call didn't exist in LLVM17, which is what clang-tidy uses