rnk accepted this revision.
rnk added a comment.
lgtm
http://reviews.llvm.org/D11932
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ismail accepted this revision.
ismail added a reviewer: ismail.
ismail added a comment.
This revision is now accepted and ready to land.
Great work. Tested fine on Windows. Next step would be mapping /openmp flag to
-fopenmp but I guess thats for another bug.
http://reviews.llvm.org/D11932
ismail added a comment.
In http://reviews.llvm.org/D11932#225158, @hfinkel wrote:
What library path should be passed (where did you build the OpenMP runtime
library)?
This is a default installation so the lib path is $PATH_TO_CLANG_EXE/../lib
which in my case is C:\Program Files\LLVM\lib
ismail added a comment.
Tests pass but, now when I tried to compile a file with -fopenmp I get:
LINK: fatal error LNK1104: cannot open file 'libomp.lib'
libomp.lib does exist in C:\Program Files\LLVM\lib but I guess we need to
pass the library path down to linker.
ABataev created this revision.
ABataev added reviewers: chandlerc, rsmith.
ABataev added a subscriber: cfe-commits.
Adds libomp.lib for -fopenmp=libomp and libiomp5md.lib for -fopenmp=libiomp5 on
Windows
http://reviews.llvm.org/D11932
Files:
lib/Driver/Tools.cpp
test/OpenMP/linking.c
ABataev updated this revision to Diff 31792.
ABataev added a comment.
Test fix
http://reviews.llvm.org/D11932
Files:
lib/Driver/Tools.cpp
test/OpenMP/linking.c
Index: lib/Driver/Tools.cpp
===
--- lib/Driver/Tools.cpp
+++