[PATCH] D78424: [flang] Added Semantic Checks for 2 Data constraints and fixed the semantic errors in 3 test cases

[Tim Keith via Phabricator via llvm-commits]

tskeith added inline comments.


================
Comment at: flang/lib/Semantics/check-data.cpp:96
+    const Symbol *symbol{name.symbol ? &name.symbol->GetUltimate() : nullptr};
+    if (IsPointer(*symbol)) { // C877
+      context_.Say(name.source,
----------------
`symbol` might be null so you need to check it here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78424