jdoerfert added a comment. In D109175#2980806 <https://reviews.llvm.org/D109175#2980806>, @weiwang wrote:
> In D109175#2980744 <https://reviews.llvm.org/D109175#2980744>, @jdoerfert > wrote: > >> Why do we need this flag, is the absence of -fopenmp-targets not sufficient? > > Just double checked, this is the full omp related options currently in use: > > "-fopenmp" > "-fopenmp-version=31" > "-fopenmp-version=31" > "-fopenmp-cuda-parallel-target-regions" > > We saw a huge number of `DECLS_TO_CHECK_FOR_DEFERRED_DIAGS` records. I don't > know if this has anything to do with omp version being 31, since prior 5.0, > everything is available on host. I don't think we are selective right now. As I was saying, disable deferred parsing if fopenmp-targets is missing, no need for this option. 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
