[PATCH] D85014: [Flang] Checks for constraint C7110-C7115.

Pete Steinfeld via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 31 09:02:50 PDT 2020


PeteSteinfeld added a comment.

In addition to my comment, I agree with all of @DavidTruby's comments.

Thanks for working on this!  I've verified that everything builds, tests and (aside from the comments in this review) looks good.



================
Comment at: flang/lib/Semantics/expression.cpp:1267-1269
+        exprAnalyzer_.Say("Value in array constructor of type '%s' could not "
+                          "be converted to the type "
+                          "of the array '%s'"_err_en_US,
----------------
This string can fit onto two lines.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85014



More information about the llvm-commits mailing list