[PATCH] D79514: [flang] Response to Steve Scalpone's suggestion to improve a message

Kiran Chandramohan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 6 13:00:44 PDT 2020


kiranchandramohan added inline comments.


================
Comment at: flang/lib/Evaluate/check-expression.cpp:210
         specExprContext_ == SpecificationExprContext::BOUND) { // C750
-      return "reference to variable '"s + symbol.name().ToString() +
-          "' not allowed for derived type components";
+      return "derived type component not allowed to reference to variable '"s +
+          symbol.name().ToString() + "'";
----------------
is there an extra "to"? Like should the message be "not allowed to reference variable"?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79514





More information about the llvm-commits mailing list