search

halide / build_bot

Buildbot configuration for Halide build bots
4 stars 6 forks source link

[Hexagon] - Build LLVM with Hexagon only if the builder handles hexagon #253

Closed pranavb-ca closed 1 year ago

pranavb-ca commented 1 year ago

This PR makes building the Hexagon backend in LLVM contingent upon the builders capability. If the builder doesn't handle hexagon there is no point building it in LLVM too.

pranavb-ca commented 1 year ago

@steven-johnson - PTAL

steven-johnson commented 1 year ago

landed and buildmaster updated

abadams commented 1 year ago

Does this affect release builds? And if so, doesn't this break the ability to cross compile for hexagon from e.g. Windows?

steven-johnson commented 1 year ago

Does this affect release builds? And if so, doesn't this break the ability to cross compile for hexagon from e.g. Windows?

Oooh, yes, this is a good point. I'll revert this.