tianshilei1992 added inline comments.
================
Comment at: clang/lib/Sema/SemaOpenMP.cpp:7249
// checking for any calls inside an Order region
if (Scope->isOpenMPOrderClauseScope())
Diag(LParenLoc, diag::err_omp_unexpected_call_to_omp_runtime_api);
----------------
ABataev wrote:
> Maybe better to check it here to avoid too early exit?
> ```
> if (Scope && Scope->...)
> ```
But does it make sense to continue? I'm not that sure.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D142233/new/
https://reviews.llvm.org/D142233
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
