gtbercea abandoned this revision.
gtbercea added a comment.
Not needed.
These changes are related to looking up the .bc library for inlining purposes.
I believe @arpith-jacob has already handled this in trunk. Therefore this is
obsolete code.
Repository:
rL LLVM
hfinkel added a comment.
In https://reviews.llvm.org/D29651#720179, @tstellar wrote:
> Why is this necessary?
To put it another way, what is special about LIBRARY_PATH in this regard? Do we
already pass paths specified with `-L`? I'm also slightly concerned here with
the custom parsing code.
ABataev added inline comments.
Comment at: test/Driver/openmp-offload.c:622
+/// Check that the lib folder pointed to by the LIBRARY_PATH is correctly
passsed to the loader script.
+// RUN: LIBRARY_PATH=/a/b/c/lib %clang -### -fopenmp=libomp
tstellar added a comment.
Why is this necessary?
Repository:
rL LLVM
https://reviews.llvm.org/D29651
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
gtbercea updated this revision to Diff 93188.
gtbercea added a comment.
Herald added a subscriber: rengolin.
Update patch to reflect latest source code changes.
Repository:
rL LLVM
https://reviews.llvm.org/D29651
Files:
lib/Driver/ToolChains/Cuda.cpp
test/Driver/openmp-offload.c