================
@@ -0,0 +1,20 @@
+// REQUIRES: host-supports-jit, x86_64-linux
+
+// RUN: rm -rf %t
+// RUN: mkdir -p %t
+//
+// RUN: split-file %s %t
+//
+// RUN: %clang++ -std=c++20 -fPIC -c %t/vec.cpp -o %t/vec.o
+// RUN: %clang++ -shared %t/vec.o -o %t/vec.so
----------------
yuxuanchen1997 wrote:

This invokes the system linker. From the build system's point of view, it's 
almost never correct to invoke the system linker without the environment 
variables `$LDFLAGS` and `$LIBS`. In this case, you expect to make the shared 
library to link against the default libs. 

I am thinking of changing this line to read 
```
// RUN: %clang++ -shared %t/vec.o -o %t/vec.so $LDFLAGS $LIBS
```
Not particularly satisfied with this approach though.

I also think that your `vec.o` and `vec.so` are both empty after my previous 
fix. This suggests that the two steps here may not even be needed at all for 
the sake of testing this fix. 

https://github.com/llvm/llvm-project/pull/117475
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to