shraiysh added inline comments.
================
Comment at: clang/lib/CodeGen/CGStmtOpenMP.cpp:5209
+ OMPLexicalScope Scope(*this, S, OMPD_unknown);
+ if (CGM.getLangOpts().OpenMPIRBuilder && S.clauses().size() == 0) {
+ llvm::OpenMPIRBuilder &OMPBuilder = CGM.getOpenMPRuntime().getOMPBuilder();
----------------
Meinersbur wrote:
> Instead of `S.clauses().size() == 0`, could we make it more explicit and
> extendable by adding a function like `isSupportedbyOpenMPIRBuilder(S)`?
Done.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D129992/new/
https://reviews.llvm.org/D129992
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
