[PATCH] D120106: [OpenMP] Add flag for disabling threat state in runtime

Johannes Doerfert via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 17 20:02:35 PST 2022


jdoerfert accepted this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.

Last nit, otherwise LG



================
Comment at: openmp/libomptarget/DeviceRTL/src/State.cpp:385
+  if (!config::mayUseThreadStates())
+    ASSERT(false && "Thread state modified while explicitly disabled!");
+
----------------
My bad, should have been
```
ASSERT(config::mayUseThreadStates() && "...").
```
also below.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120106



More information about the cfe-commits mailing list