[PATCH] D98225: [flang] Sort symbols by creation order

Pete Steinfeld via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 9 09:40:16 PST 2021


PeteSteinfeld updated this revision to Diff 329368.
PeteSteinfeld added a comment.

Responding to Tim's comments.  I moved the data member from "Symbol" to
"Symbols" and renamed it "symbolCount_".  Since "Symbols" is a template class,
this meant that I could initialize it directly in the class definition rather
than in the .cpp file.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98225

Files:
  flang/include/flang/Semantics/symbol.h
  flang/test/Semantics/data05.f90
  flang/test/Semantics/resolve102.f90
  flang/test/Semantics/typeinfo01.f90

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D98225.329368.patch
Type: text/x-patch
Size: 9589 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210309/aa6fbb92/attachment.bin>


More information about the llvm-commits mailing list