================
@@ -640,6 +642,13 @@ convertOmpOrdered(Operation &opInst, llvm::IRBuilderBase 
&builder,
   return success();
 }
 
+static LogicalResult orderedRegionSupported(omp::OrderedRegionOp op) {
----------------
skatrak wrote:

Thanks for the comment @Meinersbur. I was actually planning on creating a 
follow-up PR to centralize this a bit more, but I decided that it might help 
address your comments so I made these changes here. Yes, the idea is to have 
this be called for all operations regardless of how many unimplemented clauses 
they have, for consistency. Hopefully the template approach I updated this PR 
with works for you.

https://github.com/llvm/llvm-project/pull/114037
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to