[PATCH] D101330: [flang] Handle structure constructors with forward references to PDTs

Peter Klausler via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 27 10:00:28 PDT 2021


klausler accepted this revision.
klausler added a comment.
This revision is now accepted and ready to land.

Looks great apart from my naming nit.



================
Comment at: flang/include/flang/Semantics/expression.h:385
   }
+  bool IsBadForwardReference(const semantics::DerivedTypeSpec &);
 
----------------
Since this member function can emit a Message, it's not just a predicate; a name with "Check" would be better.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101330



More information about the llvm-commits mailing list