[PATCH] D100888: [WebAssembly] Update README. NFC.

Derek Schuff via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 20 13:57:56 PDT 2021


dschuff accepted this revision.
dschuff added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/Target/WebAssembly/README.txt:3
 
-This WebAssembly backend is presently under development.
-
-The most notable feature which is not yet stable is the ".o" file format.
-".o" file support is needed for many common ways of using LLVM, such as
-using it through "clang -c", so this backend is not yet considered widely
-usable. However, this backend is usable within some language toolchain
-packages:
-
 Emscripten provides a C/C++ compilation environment that includes standard
 libraries, tools, and packaging for producing WebAssembly applications that
----------------
You might also mention that emscripten actually contains clang as well.


================
Comment at: llvm/lib/Target/WebAssembly/README.txt:34
-
-Work-in-progress documentation for the ".o" file format is here:
-
----------------
It might be worth keeping this link (to Linking.md) in here, and maybe the C ABI doc as well?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100888



More information about the llvm-commits mailing list