[PATCH] D157994: [OpenMP] Migrate dispatch related utility functions from Clang codegen to OMPIRBuilder

[Akash Banerjee via Phabricator via cfe-commits]

TIFitis added inline comments.


================
Comment at: llvm/include/llvm/Frontend/OpenMP/OMPKinds.def:532
                        MemoryAttr(MemoryEffects::argMemOnly() | MemoryEffects::inaccessibleMemOnly(ModRefInfo::Ref)))
-        : AttributeSet(EnumAttr(NoUnwind)))
 __OMP_ATTRS_SET(
----------------
Clang test //llvm/clang/test/OpenMP/ordered_codegen.cpp// complains about the presence of `NoUnwind` attribute for the `__kmpc_dispatch_*` functions.

This change doesn't seem to break any other test, although I am not sure if we should be creating a new `Attr` group for the `__kmpc_dispatch` functions and remove the `NoUnwind` `Attr` only there.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157994