[PATCH] D112890: headers: optionalise some generated resource headers

Thorsten via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Oct 31 14:51:55 PDT 2021


tschuett added a comment.

In D112890#3099258 <https://reviews.llvm.org/D112890#3099258>, @etcwilde wrote:

> Right, the header is only necessary when you're targeting riscv, so if compiling for riscv isn't supported by the compiler noted in the `LLVM_TARGETS_TO_BUILD`, you don't need it and can't use it anyway.
> This is separate from the host platform that the compiler is running on.

I would instead say that the header is only necessary and available when you're targeting riscv. If you are targeting ARM, the header is neither available nor useful.


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