[PATCH] D78008: [flang] Added Semantic Checks for 5 Data Constraints

[Anchu Rajendran S via Phabricator via llvm-commits]

anchu-rajendran marked an inline comment as done.
anchu-rajendran added inline comments.


================
Comment at: flang/lib/Semantics/check-data.cpp:15-16
+  evaluate::ExpressionAnalyzer exprAnalyzer{context_};
+  if (MaybeExpr checked{exprAnalyzer.Analyze(x)}) {
+    if (!evaluate::IsConstantExpr(*checked)) { // C875,C881
+      context_.Say(parser::FindSourceLocation(x),
----------------
@klausler , I have made the checks on analyzed expressions as suggested by you. Please let me know if there are any concerns. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78008