JonChesterfield added subscribers: jdoerfert, JonChesterfield.
JonChesterfield added a comment.
Herald added subscribers: sstefan1, guansong, yaxunl.
Herald added a reviewer: jdoerfert.
@jdoerfert suggested replacing this with a context selector. One less special
case in clang.
Related, if this
This revision was automatically updated to reflect the committed changes.
Closed by commit rL316001: [OpenMP] Implement omp_is_initial_device() as
builtin (authored by Hahnfeld).
Changed prior to commit:
https://reviews.llvm.org/D38968?vs=119190&id=119320#toc
Repository:
rL LLVM
https://rev
Hahnfeld added a comment.
In https://reviews.llvm.org/D38968#898951, @grokos wrote:
> Now that this issue has been addressed and regressions tests pass, should we
> re-enable Cmake to build libomptarget by default?
Yes, I already have a local patch which also takes care of restricting the
tes
grokos added a comment.
Now that this issue has been addressed and regressions tests pass, should we
re-enable Cmake to build libomptarget by default?
https://reviews.llvm.org/D38968
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://li
ABataev accepted this revision.
ABataev added a comment.
This revision is now accepted and ready to land.
LG
https://reviews.llvm.org/D38968
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c
Hahnfeld created this revision.
This allows to return the static value that we know at compile time.
https://reviews.llvm.org/D38968
Files:
include/clang/Basic/Builtins.def
include/clang/Basic/Builtins.h
lib/AST/ExprConstant.cpp
lib/Basic/Builtins.cpp
test/OpenMP/is_initial_device.c
