This is an archive of the discontinued LLVM Phabricator instance.

Prefer ranlib mode over ar mode.
ClosedPublic

Authored by ed on Oct 27 2015, 9:12 AM.

Details

Summary

For CloudABI's toolchain I have a symlink that goes from <target>-ar and
<target>-ranlib to LLVM's ar binary, to mimick GNU Binutils' naming
scheme. The problem is that if we're targetting ARM64, the name of the
ranlib executable is aarch64-unknown-cloudabi-ranlib. This already
contains the string "ar".

Let's move the "ranlib" test above the "ar" test. It's not that likely
that we're going to see operating systems or harwdare architectures that
are called "ranlib".

Diff Detail

Repository
rL LLVM

Event Timeline

ed updated this revision to Diff 38551.Oct 27 2015, 9:12 AM
ed retitled this revision from to Prefer ranlib mode over ar mode..
ed updated this object.
ed added a reviewer: rafael.
rafael edited edge metadata.Oct 27 2015, 9:38 AM
rafael added a subscriber: rafael.

LGTM.

This revision was automatically updated to reflect the committed changes.