[clang] [clang][Darwin] Remove legacy framework search path logic in the frontend (PR #120149)

Louis Dionne via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 16 13:41:14 PST 2024


================
@@ -1,13 +1,6 @@
-// RUN: %clang -cc1 -fcuda-is-device -isysroot /var/empty \
-// RUN:   -triple nvptx-nvidia-cuda -aux-triple i386-apple-macosx \
-// RUN:   -E -fcuda-is-device -v -o /dev/null -x cuda %s 2>&1 | FileCheck %s
-
-// RUN: %clang -cc1 -isysroot /var/empty \
-// RUN:   -triple i386-apple-macosx -aux-triple nvptx-nvidia-cuda \
-// RUN:   -E -fcuda-is-device -v -o /dev/null -x cuda %s 2>&1 | FileCheck %s
-
 // Check that when we do CUDA host and device compiles on MacOS, we check for
 // includes in /System/Library/Frameworks and /Library/Frameworks.
 
-// CHECK-DAG: ignoring nonexistent directory "/var/empty/System/Library/Frameworks"
-// CHECK-DAG: ignoring nonexistent directory "/var/empty/Library/Frameworks"
+// RUN: %clang -isysroot /var/empty -target unknown-nvidia-cuda -v -fsyntax-only -x cuda %s -### 2>&1 | FileCheck %s
----------------
ldionne wrote:

@jhuber6 Pinging you for Cuda questions, feel free to let me know if I should be asking someone else instead.

This test doesn't pass with the current patch. It looks like the Cuda driver isn't calling the Darwin toolchain driver when setting up search paths. Previously this was all done in the frontend so that wasn't a problem, but I don't understand how that's supposed to work. Do you have familiarity with this part of Clang?

https://github.com/llvm/llvm-project/pull/120149


More information about the cfe-commits mailing list