[PATCH] D142688: [Clang][Driver] Handle LoongArch multiarch tuples

Wed Apr 19 05:32:19 PDT 2023

SixWeining accepted this revision.
SixWeining added inline comments.
This revision is now accepted and ready to land.

================
Comment at: clang/test/Driver/linux-header-search.cpp:247
 //
+// Check header search on Debian loong64
+// RUN: %clang -### %s -fsyntax-only 2>&1 \
----------------
xen0n wrote:
> SixWeining wrote:
> > Better to use “loongarch64” when talking about debian? Refer https://wiki.debian.org/Multiarch/Tuples
> It's the dpkg arch identifier and actually [[ https://wiki.debian.org/Ports/loong64 | the port's official name ]]. I chose it because it's shorter. Do we still want to emphasize the target tuple instead of the distro identity in this case?
Sorry I didn’t noticed this before. Now it LGTM. Thanks.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142688