[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