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

[Valentin Clement via Phabricator via llvm-commits]

clementval added inline comments.


================
Comment at: flang/lib/Semantics/check-omp-structure.cpp:13
 #include <algorithm>
-
 namespace Fortran::semantics {
----------------
unrelated change?


================
Comment at: flang/test/Semantics/omp-sync-critical02.f90:17
+
+end function timer_tick_sec
----------------
I'm not sure I follow what this test adds to the same code in `omp-sync-critical01.f90`...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110502