[PATCH] D110663: [Driver] Support Debian multiarch style lib/clang/14.0.0/x86_64-linux-gnu runtime path and include/x86_64-linux-gnu/c++/v1 libc++ path

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 28 18:07:37 PDT 2021


MaskRay added a comment.

I'd actually want to get rid of the function `Linux::getMultiarchTriple`, when all Debian Clang use `LLVM_ENABLE_PER_TARGET_RUNTIME_DIR=*-linux-gnu` instead of `LLVM_ENABLE_PER_TARGET_RUNTIME_DIR=*-{unknown,pc}-linux-gnu`

In D110663#3029131 <https://reviews.llvm.org/D110663#3029131>, @jrtc27 wrote:

> x86 is unusual; the default vendor, at least in the GNU world, is pc, not unknown (e.g. `config.guess x86_64-linux-gnu` prints x86_64-pc-linux-gnu).

OK. llvm-project's old config.guess uses `unknown`.

  % sh llvm/cmake/config.guess       
  x86_64-unknown-linux-gnu
  % sh /tmp/c/config.guess   # 'https://git.savannah.gnu.org/gitweb/?p=config.git;a=blob_plain;f=config.guess;hb=HEAD'
  x86_64-pc-linux-gnu


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110663



More information about the cfe-commits mailing list