jdenny added a comment. In D56113#1345578 <https://reviews.llvm.org/D56113#1345578>, @ABataev wrote:
> >>> By the way, LangOpts.OpenMP currently defaults to 0. Should it? > >> > >> Yes, it means it is disabled by default. > > > > Ah, it becomes 1 when we have `-fopenmp` but not `-fopenmp-version`. But > > still `LangOpts.OpenMP <= 31` is true by default, so the new behavior would > > be off by default. > > No, by default it gets value 31. In `DSAStackTy::getTopDSA` in SemaOpenMP.cpp, I added this line: llvm::errs() << SemaRef.LangOpts.OpenMP << '\n'; `clang -fopenmp test.c` prints 1. I'm at r350238, which I pulled yesterday. For my purposes, 1 and 31 are equivalent, so I can move on. > If -fopenmp-targets= is used, then the default version is 45. That I can confirm. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56113/new/ https://reviews.llvm.org/D56113 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
