[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 llvm-commits at lists.llvm.org
Tue Apr 21 07:31:25 PDT 2020


tskeith accepted this revision.
tskeith added inline comments.
This revision is now accepted and ready to land.


================
Comment at: flang/include/flang/Semantics/tools.h:167
 }
+inline bool IsAutomaticArray(const Symbol &symbol) {
+  if (symbol.IsObjectArray()) {
----------------
I don't think there is any reason for this to be inline.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78424





More information about the llvm-commits mailing list