psoni2628 added inline comments.
================
Comment at: clang/lib/CodeGen/CGStmtOpenMP.cpp:2596
// Check for unsupported clauses
- if (!S.clauses().empty()) {
- // Currently no clause is supported
- return false;
+ for (OMPClause *C : S.clauses()) {
+ // Currently only simdlen clause is supported
----------------
arnamoy10 wrote:
> I am just wondering whether we should have a check to make sure that we are
> processing the clauses of only `simd` directive here. Because the function
> takes a general `OMPExecutableDirective` as argument
That's a fair point. I guess `isSupportedByOpenMPIRBuilder` could be used for
other directive types other than simd, even though it's not right now.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D129149/new/
https://reviews.llvm.org/D129149
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
