This is an archive of the discontinued LLVM Phabricator instance.

Differential D50156

[test] Fix %hmaptool path for standalone builds
ClosedPublic

Authored by mgorny on Aug 1 2018, 1:08 PM.

Details

Reviewers
bkramer
bruno
Commits
rGcebee05a4da4: Merging r338627: --------------------------------------------------------------…
rG624169f72e89: [test] Fix %hmaptool path for standalone builds
rL339102: Merging r338627:
rL338627: [test] Fix %hmaptool path for standalone builds
rC338627: [test] Fix %hmaptool path for standalone builds
Summary

Fix %hmaptool path to refer to clang_tools_dir instead of
llvm_tools_dir, in order to fix standalone builds. The tool is built
as part of clang, so it won't be found in installed LLVM tools.

Diff Detail

Event Timeline

mgorny created this revision.Aug 1 2018, 1:08 PM
bkramer accepted this revision.Aug 1 2018, 1:14 PM

lg, thanks

This revision is now accepted and ready to land.Aug 1 2018, 1:14 PM
Closed by commit rC338627: [test] Fix %hmaptool path for standalone builds (authored by mgorny). · Explain WhyAug 1 2018, 1:38 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents

PathSize
test/
2 lines

Diff 158612

test/lit.cfg.py

Context not available.
config.substitutions.append( config.substitutions.append(
('%hmaptool', "'%s' %s" % (config.python_executable, ('%hmaptool', "'%s' %s" % (config.python_executable,
os.path.join(config.llvm_tools_dir, 'hmaptool')))) os.path.join(config.clang_tools_dir, 'hmaptool'))))
# Plugins (loadable modules) # Plugins (loadable modules)
# TODO: This should be supplied by Makefile or autoconf. # TODO: This should be supplied by Makefile or autoconf.
Context not available.