[PATCH] D135431: Extended Documentation of LLVM_TARGETS_TO_BUILD by listing all possible values

[Kristof Beyls via Phabricator via llvm-commits]

kristof.beyls added a comment.

I just have one minor suggestion.
Otherwise, LGTM.



================
Comment at: llvm/docs/CMake.rst:741
   targets. Case-sensitive. Defaults to *all*. Example:
-  ``-DLLVM_TARGETS_TO_BUILD="X86;PowerPC"``.
+  ``-DLLVM_TARGETS_TO_BUILD="X86;PowerPC"``. The full list is:
+  ``AArch64;AMDGPU;ARM;AVR;BPF;Hexagon;Lanai;Mips;MSP430;NVPTX;PowerPC;RISCV;Sparc;SystemZ;VE;WebAssembly;X86;XCore``
----------------
The backends available in LLVM can change over time.
When a backend is removed/added, the people doing so may well forget to update the list here.
Therefore, maybe it may be better to write:
"The full list, as of October 2022, is:"?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D135431/new/

https://reviews.llvm.org/D135431