weiwang added a comment. In D109175#2987048 <https://reviews.llvm.org/D109175#2987048>, @jdoerfert wrote:
> In D109175#2986782 <https://reviews.llvm.org/D109175#2986782>, @yaxunl wrote: > >> I agree with Johannes and Alexey that deferred diags are only needed when >> LangOpts.OMPTargetTriples.empty(). However, I am not sure whether it is only >> needed in device compilation. >> >> For other offloading languages like CUDA/HIP it is needed in both device and >> host compilation. > > Technically, we might even want to delay in host only mode for OpenMP, but > that is something we can revisit (e.g., by dynamically setting a flag based > on the directives we've seen). > @yaxunl Should we for now check if there is any associated offload job? Shall we go ahead and get this change in and think about more longer term solution later? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109175/new/ https://reviews.llvm.org/D109175 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
