[PATCH] D85617: [OpenMP][OMPBuilder] Adding support for `omp single`

[Fady Ghanim via Phabricator via llvm-commits]

fghanim marked an inline comment as done.
fghanim added inline comments.


================
Comment at: llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp:811
+  if (DidIt) {
+    Builder.CreateStore(Builder.getInt32(0), DidIt);
+  }
----------------
kiranchandramohan wrote:
> Where will this variable be updated to 1 to show it is the single thread?
Inside the `.... single region...` below. Check line 3682 in D85619 to see actual update


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85617