[PATCH] D93051: [Flang][openmp] Add semantic checks for OpenMP critical construct.

[Tim Keith via Phabricator via llvm-commits]

tskeith added inline comments.


================
Comment at: flang/lib/Semantics/resolve-directives.cpp:1065
+  const parser::Name *mayBeName{&name};
+  if (mayBeName) {
+    if (auto *resolvedSymbol{
----------------
What is the purpose of introducing `maybeName` when you can just use `name`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93051