This is an archive of the discontinued LLVM Phabricator instance.

[docs] Copy-edit lld/docs/WebAssembly.rst
ClosedPublic

Authored by siedentop on Apr 22 2019, 6:16 PM.

Details

Summary

Fixes small typos in WebAssembly documentation. I first noticed the sub-heading "Bahavior", and then decided to review the whole file.

Diff Detail

Event Timeline

siedentop created this revision.Apr 22 2019, 6:16 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 22 2019, 6:16 PM
sbc100 accepted this revision.Apr 23 2019, 7:22 AM

Great! Thank you.

lld/docs/WebAssembly.rst
13

How about The WebAssemby object file format used by LLVM and LLD is specified as ...

19

This last sentence is no longer true. WebAssembly is on the of he default targets now (Not part of this PR but something I just noticed).

This revision is now accepted and ready to land.Apr 23 2019, 7:22 AM
siedentop updated this revision to Diff 196361.Apr 23 2019, 6:28 PM

Thank you for the suggestion. Updated as suggested.

sbc100 accepted this revision.Apr 23 2019, 10:36 PM
siedentop marked 2 inline comments as done.Apr 24 2019, 8:04 AM

Could someone please merge/commit this now? I do not have commit access. First contribution :)

This revision was automatically updated to reflect the committed changes.