This is an archive of the discontinued LLVM Phabricator instance.

HIP: Try to deal with more llvm package layouts
ClosedPublic

Authored by arsenm on May 22 2020, 4:47 PM.

Details

Reviewers
yaxunl
scchan
Summary

The various HIP builds are all inconsistent.

The default llvm install goes to ${INSTALL_PREFIX}/bin/clang, but the
rocm packaging scripts move this under
${INSTALL_PREFIX}/llvm/bin/clang. Some other builds further pollute
this with ${INSTALL_PREFIX}/bin/x86_64/clang. These should really be
consolidated, but try to handle them for now.

Diff Detail

Event Timeline

arsenm created this revision.May 22 2020, 4:47 PM
yaxunl accepted this revision.May 22 2020, 8:22 PM

LGTM. Thanks.

This revision is now accepted and ready to land.May 22 2020, 8:22 PM