[Openmp-commits] [PATCH] D51694: Fix for https://bugs.llvm.org/show_bug.cgi?id=38839

[Paul Osmialowski via Phabricator via Openmp-commits]

pawosm01 added inline comments.


================
Comment at: runtime/src/include/50/omp_lib.h.var:31
+      integer(kind=omp_integer_kind),parameter::kmp_version_minor=@LIBOMP_VERSION_MINOR@
+      integer(kind=omp_integer_kind),parameter::                        &
+     &kmp_version_build=@LIBOMP_VERSION_BUILD@
----------------
It's not gonna work if someone use fixed format and extended line len, compiler will fail on this & at end.
There are omp_lib.h implementation that are
1) fixed format
2) yet still have lines no longer than 72 characters
so they can be always included, e.g.:
https://github.com/flang-compiler/flang/blob/master/include/omp_lib.h



Repository:
  rOMP OpenMP

https://reviews.llvm.org/D51694