[PATCH] D79798: [flang] Constraint checks C751 to C760

[Tim Keith via Phabricator via llvm-commits]

tskeith added inline comments.


================
Comment at: flang/lib/Semantics/resolve-names.cpp:3677
+      CHECK(!attrs_);
+      attrs_ = std::make_optional<Attrs>();
+      Symbol *typeParam{MakeTypeSymbol(
----------------
You can use `BeginAttrs` here instead, and `EndAttrs` below.


================
Comment at: flang/lib/Semantics/type.cpp:154
       } else {
-        messages.Say(name_,
-            "Type parameter '%s' lacks a value and has no default"_err_en_US,
-            name);
+        if (!symbol.test(Symbol::Flag::Error)) {
+          messages.Say(name_,
----------------
Could be `else if`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79798