[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