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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to