Add a testcase for riscv64-linux-android triple

[hiraditya]

No description provided.

[llvmbot]

@llvm/pr-subscribers-clang-driver

@llvm/pr-subscribers-clang

Author: AdityaK (hiraditya)

Changes

---

Full diff: https://github.com/llvm/llvm-project/pull/116892.diff

1 Files Affected:

• (modified) clang/test/Driver/android-ndk-standalone.cpp (+16)

diff --git a/clang/test/Driver/android-ndk-standalone.cpp b/clang/test/Driver/android-ndk-standalone.cpp
index 2670a6707a96c9..3d18244c77107b 100644
--- a/clang/test/Driver/android-ndk-standalone.cpp
+++ b/clang/test/Driver/android-ndk-standalone.cpp
@@ -311,3 +311,19 @@
 // CHECK-X86_64-GCC: Found candidate GCC installation: {{.*}}i686-linux-android{{[/\\]}}4.9
 // CHECK-X86_64-GCC-NEXT: Found candidate GCC installation: {{.*}}x86_64-linux-android{{[/\\]}}4.9
 // CHECK-X86_64-GCC-NEXT: Selected GCC installation: {{.*}}x86_64-linux-android{{[/\\]}}4.9
+//
+// RUN: %clang -### %s 2>&1 \
+// RUN:     --target=riscv64-linux-android \
+// RUN:     -march=rv64i \
+// RUN:     --gcc-toolchain=%S/Inputs/basic_android_ndk_tree \
+// RUN:     --sysroot=%S/Inputs/basic_android_ndk_tree/sysroot \
+// RUN:   | FileCheck --check-prefix=CHECK-RISCV %s
+// CHECK-RISCV: "-cc1"
+// CHECK-RISCV: "-internal-isystem" "{{.*}}/include/c++/v1"
+// CHECK-RISCV: "-internal-isystem" "{{.*}}/lib/clang/{{..}}/include"
+// CHECK-RISCV: "-internal-isystem" "{{.*}}/sysroot/usr/local/include"
+// CHECK-RISCV: "-internal-externc-isystem" "{{.*}}/sysroot/include"
+// CHECK-RISCV: "-internal-externc-isystem" "{{.*}}/sysroot/usr/include"
+// CHECK-RISCV: "{{.*}}ld{{(.exe)?}}" "--sysroot=[[SYSROOT:[^"]+]]"
+// CHECK-RISCV: "-L{{.*}}/sysroot/usr/lib"
+// CHECK-RISCV: "{{.*}}/lib/riscv64-unknown-linux-android/libclang_rt.builtins.a"

[pirama-arumuga-nainar]

Will defer to @DanAlbert. I am not sure if basic_android_ndk_tree is relevant for this test - if so, we should add riscv64 components to that subdir.

[DanAlbert]

The CHECK-ARM64 variant of the test has it. I think it's so the test can find the linker? As long as this test is essentially a copy of that with the target and arch names swapped out, it's at least not wronger than the existing tests.

[pirama-arumuga-nainar]

These should just be ld to match what is in clang/test/Driver/Inputs/basic_android_ndk_tree/arm-linux-androideabi/bin/. @DanAlbert and I discussed whether we can assume lld is the default since the Android toolchain defaults to lld but that's still undecided.