[PATCH] D103125: [Clang][WIP] Allow renaming of "clang"

Paul Robinson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 27 05:34:03 PDT 2021


probinson added a comment.

In D103125#2782096 <https://reviews.llvm.org/D103125#2782096>, @echristo wrote:

> I'm also not a fan of this change. From a project perspective the binary is clang and while people may wish to change the name for their own product teams it seems like that onus should be on them.

It occurs to me that there's precedent regarding prefixes, which is that you can rename/soft-link clang to have a triple embedded in the name (e.g., x86_64-pc-linux-clang) and then the driver will factor the prefix into default-target computations.  But AFAIK there's no support for directly doing that renaming/linking in the build system.  I agree with @echristo this properly belongs to the vendors not the project.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D103125/new/

https://reviews.llvm.org/D103125



More information about the llvm-commits mailing list