ABataev added a comment. In D56113#1345588 <https://reviews.llvm.org/D56113#1345588>, @jdenny wrote:
> 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. Ah, yes, forgot already. You can consider 1 as the same value as 31 by default. > > >> 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