[PATCH] D110502: [Flang][openmp] Add semantic checks for OpenMP critical construct name resolution

Wed Sep 29 07:18:16 PDT 2021

NimishMishra added inline comments.

================
Comment at: flang/lib/Semantics/check-omp-structure.cpp:13
 #include <algorithm>
-
 namespace Fortran::semantics {
----------------
clementval wrote:
> unrelated change?
Not sure what's caused this. Could it be possible that git-clang-format introduced these?

================
Comment at: flang/test/Semantics/omp-sync-critical02.f90:17
+
+end function timer_tick_sec
----------------
clementval wrote:
> I'm not sure I follow what this test adds to the same code in `omp-sync-critical01.f90`...
I added this verbatim from the artifact at https://bugs.llvm.org/show_bug.cgi?id=48145. Should I remove it?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110502