[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

Tue, 28 Sep 2021 18:07:43 -0700 

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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to