clementval added inline comments.

================
Comment at: flang/test/Driver/driver-help.f90:55
 ! HELP-NEXT: -fopenmp-version=<value>
-! HELP-NEXT:                        Set OpenMP version (e.g. 45 for OpenMP 
4.5, 50 for OpenMP 5.0). Default value is 50 for Clang and 11 for Flang
+! HELP-NEXT:                        Set OpenMP version (e.g. 45 for OpenMP 
4.5, 51 for OpenMP 5.1). Default value is 51 for Clang and 11 for Flang
 ! HELP-NEXT: -fopenmp               Parse OpenMP pragmas and generate parallel 
code.
----------------
jdoerfert wrote:
> tianshilei1992 wrote:
> > clementval wrote:
> > > AntonRydahl wrote:
> > > > tianshilei1992 wrote:
> > > > > Does Flang also switch to 5.1 by default?
> > > > As far as I know, `flang` is still OpenMP 1.1 by default but still 
> > > > depends on the same command line options as `clang`. 
> > > `flang` does not differentiate between versions. 
> > I see.
> I would remove the flang part of the wording for now.
+1


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D156901/new/

https://reviews.llvm.org/D156901

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to