[PATCH] D79347: [flang] Handle EQUIVALENCE and COMMON in size and offset computations

[Steve Scalpone via Phabricator via llvm-commits]

sscalpone accepted this revision.
sscalpone added inline comments.
This revision is now accepted and ready to land.


================
Comment at: flang/include/flang/Semantics/symbol.h:306
   void set_bindName(MaybeExpr &&expr) { bindName_ = std::move(expr); }
+  std::size_t align() const { return align_; }
+  void set_align(std::size_t align) { align_ = align; }
----------------
align is a verb, and someone might think this call is aligning something.  how about g/align/s//alignment/?


================
Comment at: flang/lib/Semantics/compute-offsets.cpp:74
 
+static bool InCommonBlock(const Symbol &symbol) {
+  const auto *details{symbol.detailsIf<ObjectEntityDetails>()};
----------------
InCommonBlock might be helpful for lowering, but i'm just guessing, because i'm not doing any lowering.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79347