[PATCH] D38968: [OpenMP] Implement omp_is_initial_device() as builtin

Jon Chesterfield via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 6 04:16:50 PST 2021


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 is a builtin (or in header), I think we can and should delete it from the devicertl. It's probably used as a branch condition so replacing with a 0/1 asap is winning for compilation time.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D38968/new/

https://reviews.llvm.org/D38968



More information about the cfe-commits mailing list