LLVM Bugzilla is read-only and represents the historical archive of all LLVM issues filled before November 26, 2021. Use github to submit LLVM bugs

Bug 38407 - clang: tests unable to find hmaptool in stand-alone builds
Summary: clang: tests unable to find hmaptool in stand-alone builds
Status: RESOLVED FIXED
Alias: None
Product: clang
Classification: Unclassified
Component: -New Bugs (show other bugs)
Version: 7.0
Hardware: PC Linux
: P enhancement
Assignee: Unassigned Clang Bugs
URL:
Keywords:
Depends on:
Blocks: release-7.0.0
  Show dependency tree
 
Reported: 2018-08-01 13:13 PDT by Michał Górny
Modified: 2018-08-06 23:58 PDT (History)
2 users (show)

See Also:
Fixed By Commit(s): r338627 r339102


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michał Górny 2018-08-01 13:13:12 PDT
The following clang tests fail in stand-alone builds:

********************
Failing Tests (5):
    Clang :: Modules/double-quotes.m
    Clang :: Modules/framework-public-includes-private.m
    Clang :: Preprocessor/headermap-rel.c
    Clang :: Preprocessor/headermap-rel2.c
    Clang :: Preprocessor/nonportable-include-with-hmap.c


The issue is wrong path to hmaptool:

Command Output (stderr):
--
+ : 'RUN: at line 3'
+ rm -rf /var/tmp/portage/sys-devel/clang-7.0.9999/work/x/y/clang-7.0.9999-abi_x86_64.amd64/test/Modules/Output/double-quotes.m.tmp
+ : 'RUN: at line 4'
+ mkdir /var/tmp/portage/sys-devel/clang-7.0.9999/work/x/y/clang-7.0.9999-abi_x86_64.amd64/test/Modules/Output/double-quotes.m.tmp
+ : 'RUN: at line 6'
+ /var/tmp/portage/sys-devel/clang-7.0.9999/temp/python2.7/bin/python /usr/lib64/llvm/7/bin/hmaptool write /var/tmp/portage/sys-devel/clang-7.0.9999/work/x/y/clang-7.0.9999/test/Modules/Inputs/double-quotes/a.hmap.json /var/tmp/portage/sys-devel/clang-7.0.9999/work/x/y/clang-7.0.9999-abi_x86_64.amd64/test/Modules/Output/double-quotes.m.tmp/a.hmap
/usr/bin/python2.7: can't open file '/usr/lib64/llvm/7/bin/hmaptool': [Errno 2] No such file or directory

--

I've submitted a potential fix as https://reviews.llvm.org/D50156.  Once it's committed to trunk, please backport to release_70.
Comment 1 Michał Górny 2018-08-01 13:40:20 PDT
Now in trunk as r338627.
Comment 2 Hans Wennborg 2018-08-06 23:58:13 PDT
(In reply to Michał Górny from comment #1)
> Now in trunk as r338627.

Merged in r339102.