[PATCH] D112890: headers: optionalise some generated resource headers
Reid Kleckner via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Nov 1 10:44:16 PDT 2021
rnk added a comment.
I think this seems reasonable.
Generally speaking, we have tests in clang that generate IR for targets that are not enabled in LLVM. It is kind of nice: you don't have to mark the IRGen tests with `REQUIRES: foo-registered-target`. If we want to test these ARM and RISCV intrinsic tests in clang, now, they will have to be conditioned on the relevant target being enabled. I think that's fine.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D112890/new/
https://reviews.llvm.org/D112890
More information about the cfe-commits
mailing list