================
@@ -706,6 +706,32 @@ static bool supportsSPMDExecutionMode(ASTContext &Ctx,
       "Unknown programming model for OpenMP directive on NVPTX target.");
 }
 
+static bool isNoLoopEligible(ASTContext &Ctx, const OMPExecutableDirective &D) 
{
----------------
dhruvachak wrote:

This method is not expensive, so I don't see the need to cache. Using a cache 
hosted by the runtime for the current directive is non-intuitive and not 
future-proof. Please go back to the previous design.

https://github.com/llvm/llvm-project/pull/224041
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to