weiwang added a comment.

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 and `getEmissionStatus` would return 
`FunctionEmissionStatus::Unknown` in this case.


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

Reply via email to