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

[Andrey Churbanov via Phabricator via Openmp-commits]

AndreyChurbanov added inline comments.


================
Comment at: runtime/src/include/50/omp_lib.h.var:28-30
+      integer(kind=omp_integer_kind),parameter::openmp_version=@LIBOMP_OMP_YEAR_MONTH@
+      integer(kind=omp_integer_kind),parameter::kmp_version_major=@LIBOMP_VERSION_MAJOR@
+      integer(kind=omp_integer_kind),parameter::kmp_version_minor=@LIBOMP_VERSION_MINOR@
----------------
jlpeyton wrote:
> Aren't these longer than 72 chars?
Yes - before variables substitution, and NO - after the substitution performed during the build.
LIBOMP_OMP_YEAR_MONTH - 6 chars, LIBOMP_VERSION* - single char each.
So the built header has these lines short enough.


Repository:
  rOMP OpenMP

https://reviews.llvm.org/D51694