This is an archive of the discontinued LLVM Phabricator instance.

[WebAssembly] Fix location and -flavor when running lld
ClosedPublic

Authored by sbc100 on May 8 2017, 12:57 PM.

Details

Summary

Add the toolchain installation directory to the program
path so that lld can be found.

Update -flavor to be wasm. Although this new flavor
hasn't yet landed in upstream lld yet there are no point
in passing wasm objects the gnu flavor.

Diff Detail

Repository
rL LLVM

Event Timeline

sbc100 created this revision.May 8 2017, 12:57 PM
sbc100 edited the summary of this revision. (Show Details)May 8 2017, 2:07 PM
dschuff accepted this revision.May 9 2017, 9:36 AM
This revision is now accepted and ready to land.May 9 2017, 9:36 AM
This revision was automatically updated to reflect the committed changes.
pcc added a subscriber: pcc.May 9 2017, 12:09 PM

Hi Sam, just out of curiosity, are there any details available about your plans for the wasm port of lld? I couldn't seem to find any by searching the lists.

sbc100 added a comment.May 9 2017, 1:08 PM

Hi Peter. I've been doing some preliminary work which is currently here: https://github.com/sbc100/lld/tree/add_wasm_linker_skeleton

I have been meaning to send out a message about this work to the mailing list, and this is probably good time to do that. As soon as the prototype is able to link basic single-file programs I was hoping to try and get it merged upstream. There is a general lack of a real linker story in the wasm world right now and lld seemed like to good place to build one.