[PATCH] D92491: [flang][NFC] Add GetTopLevelUnitContaining functions

[Peter Klausler via Phabricator via llvm-commits]

klausler accepted this revision.
klausler added inline comments.


================
Comment at: flang/include/flang/Semantics/tools.h:33
 
+// Note: Here ProgramUnit includes internal subprograms while TopLevelUnit
+// does not. "program-unit" in the Fortran standard matches TopLevelUnit.
----------------
"top level unit" is self-defining and I like it.  Can we avoid "program unit" altogether, though?  (It can't be "subprogram", either.) 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92491