This is an archive of the discontinued LLVM Phabricator instance.

[WebAssembly] Rename relocations from R_WEBASSEMBLY_ to R_WASM_. NFC
ClosedPublic

Authored by sbc100 on Feb 1 2019, 12:31 PM.

Details

Summary

See https://github.com/WebAssembly/tool-conventions/pull/95.

This is less typing and IMHO more readable, and it also fits with
our naming around the binary format which tends to use the short name.
e.g.

include/llvm/BinaryFormat/Wasm.h
tools/llvm-objdump/WasmDump.cpp
etc..

Diff Detail

Event Timeline

sbc100 created this revision.Feb 1 2019, 12:31 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 1 2019, 12:31 PM
sbc100 retitled this revision from [WebAssembly] Rename relocations from R_WEBASSEMBLY_ to R_WASM_ to [WebAssembly] Rename relocations from R_WEBASSEMBLY_ to R_WASM_. NFC.Feb 1 2019, 12:31 PM
sbc100 added a reviewer: sunfish.
This revision was not accepted when it landed; it landed in state Needs Review.Feb 4 2019, 9:28 AM
This revision was automatically updated to reflect the committed changes.