[PATCH] D127645: [docs] Adding table of object file formats

[Fangrui Song via Phabricator via llvm-commits]

MaskRay accepted this revision.
MaskRay added a comment.
This revision is now accepted and ready to land.

Some suggestions inlined



================
Comment at: llvm/docs/CodeGenerator.rst:711
+The MC layer's object writers support a variety of object formats. Because of
+target-specific aspects of object formats each target only supports a subset of
+the formats supported by the MC layer. Most targets support emitting ELF
----------------
object file formats


================
Comment at: llvm/docs/CodeGenerator.rst:714
+objects. Other vendor-specific objects are generally supported only on targets
+that are supported by that vendor (i.e. MachO is only supported on targets
+supported by Darwin, and XCOFF is only supported on targets that support AIX).
----------------
I'd delete the `i.e.` sentence, since the information is redundant given the table below.


================
Comment at: llvm/docs/CodeGenerator.rst:731
+     ``MachO``           AArch64, ARM, X86
+     ``SPIR-V``          SPIRV
+     ``WASM``            WebAssembly
----------------
Use SPIRV instead of SPIR-V.

(Otherwise you need to use Mach-O instead of MachO).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127645