This is an archive of the discontinued LLVM Phabricator instance.

Differential D43417

[WebAssembly] Add wasm-ld to lit tool_patterns
AbandonedPublic

Authored by sbc100 on Feb 16 2018, 3:20 PM.

Details

Reviewers
ruiu
Summary

Without this lit doesn't print/use the full path the executable.
Although this doesn't seem to break the test it does make copying
and pasting the commands from lit more difficult.

Diff Detail

Event Timeline

sbc100 created this revision.Feb 16 2018, 3:20 PM
Herald added subscribers: llvm-commits, delcypher, sunfish and 4 others. · View Herald TranscriptFeb 16 2018, 3:20 PM
sbc100 added a reviewer: ruiu.Feb 16 2018, 3:20 PM
Harbormaster completed remote builds in B15112: Diff 134743.Feb 16 2018, 3:20 PM
ruiu added a comment.Feb 16 2018, 3:21 PM

Actually I've just did that a few hours ago. Please sync to SVN head.

sbc100 abandoned this revision.Feb 16 2018, 3:33 PM

Ha!

Revision Contents

PathSize
utils/
lit/
lit/
llvm/
2 lines

Diff 134743

utils/lit/lit/llvm/config.py

Show First 20 Lines • Show All 462 Lines • ▼ Show 20 Lines def use_lld(self, required=True):
lib_dirs = lib_dirs + [lld_libs_dir] lib_dirs = lib_dirs + [lld_libs_dir]
self.with_environment('PATH', tool_dirs, append_path=True) self.with_environment('PATH', tool_dirs, append_path=True)
self.with_environment('LD_LIBRARY_PATH', lib_dirs, append_path=True) self.with_environment('LD_LIBRARY_PATH', lib_dirs, append_path=True)
self.config.substitutions.append( self.config.substitutions.append(
(r"\bld.lld\b", 'ld.lld --full-shutdown')) (r"\bld.lld\b", 'ld.lld --full-shutdown'))
tool_patterns = ['ld.lld', 'lld-link', 'lld'] tool_patterns = ['ld.lld', 'lld-link', 'lld', 'wasm-ld']
self.add_tool_substitutions(tool_patterns, tool_dirs) self.add_tool_substitutions(tool_patterns, tool_dirs)