[flang-commits] [PATCH] D109905: [flang] More precise checks for NULL() operands

[Pete Steinfeld via Phabricator via flang-commits]

PeteSteinfeld accepted this revision.
PeteSteinfeld added a comment.
This revision is now accepted and ready to land.

All looks good.



================
Comment at: flang/lib/Semantics/check-omp-structure.cpp:55-62
+    if (lhs && rhs) {
+      Tristate isDefined{semantics::IsDefinedAssignment(
+          lhs->GetType(), lhs->Rank(), rhs->GetType(), rhs->Rank())};
+      if (isDefined == Tristate::Yes) {
+        context_.Say(expr.source,
+            "Defined assignment statement is not "
+            "allowed in a WORKSHARE construct"_err_en_US);
----------------
This change doesn't seem related to everything else (although it looks good).  Did you mean to include it?


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

https://reviews.llvm.org/D109905