This is an archive of the discontinued LLVM Phabricator instance.

[WebAssembly] Update README. NFC.
ClosedPublic

Authored by sbc100 on Apr 20 2021, 1:48 PM.

Details

Summary

This is just a cleanup of the very high level stuff. I'm sure there is
more to update here but I'll leave that to others and/or a followup.

Diff Detail

Unit TestsFailed

Event Timeline

sbc100 created this revision.Apr 20 2021, 1:48 PM
sbc100 requested review of this revision.Apr 20 2021, 1:48 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 20 2021, 1:48 PM
dschuff accepted this revision.Apr 20 2021, 1:57 PM
dschuff added inline comments.
llvm/lib/Target/WebAssembly/README.txt
3

You might also mention that emscripten actually contains clang as well.

34

It might be worth keeping this link (to Linking.md) in here, and maybe the C ABI doc as well?

This revision is now accepted and ready to land.Apr 20 2021, 1:57 PM
dschuff added inline comments.Apr 20 2021, 1:58 PM
llvm/lib/Target/WebAssembly/README.txt
28

oh also delete this section.

sbc100 marked 3 inline comments as done.Apr 20 2021, 4:58 PM
sbc100 updated this revision to Diff 339056.Apr 20 2021, 4:58 PM
  • feedback
This revision was landed with ongoing or failed builds.Apr 20 2021, 4:59 PM
This revision was automatically updated to reflect the committed changes.