[Mlir-commits] [llvm] [mlir] Config: Move LLVM_HAS_*_TARGET definitions to a new header. (PR #136388)

[Fangrui Song]

MaskRay wrote:

> I'm strongly in favor of this change. I often enable/disable targets and it results in rebuilding everything. ccache helps, but it still runs the preprocessing step (I think), which isn't quite fast. Also, any change in target list results in llvm-tblgen trying to rebuild all the generated files, which is even slower.

+1.

The rebuild issue has been raised on the original change that added the macros: https://github.com/llvm/llvm-project/pull/71164
Thanks for taking care of it :)

https://github.com/llvm/llvm-project/pull/136388