search

halide / Halide

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

Remove Hexagon runtime prebuilts #3096

Open dsharletg opened 6 years ago

dsharletg commented 6 years ago

We shouldn't be checking in prebuilts. Instead, we should have the buildbots build them, and distribute them as part of Halide releases instead.

ronlieb commented 6 years ago

sounds like a great idea

ronlieb commented 6 years ago

as i recall, the issue we ran into before, is that we have dependencies on having an installed HEXAGON_SDK. The qaic compiler, elfsigner, include files and libs are all needed to rebuild these libraries. Folks who want to use hexagon as a target are able to do so because of the committed prebuilts.

dsharletg commented 5 years ago

I had another idea recently: we could just "prebuild" the output of the RPC compiler, which is just some C code (not a binary) so it can be diffed and such when modifying the RPC definition. The rest could be built as part of the normal halide build as I don't think it would have any dependencies on the Hexagon SDK.