[PATCH] D108130: [flang][nfc] Tweak the FrontendAction class

Asher Mancinelli via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 16 10:14:57 PDT 2021


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


================
Comment at: flang/lib/Frontend/FrontendAction.cpp:159
+
+bool FrontendAction::RunSema() {
+  CompilerInstance &ci = this->instance();
----------------
[nit] Would it be too invasive to expand this name in this patch, eg `RunSemanticAction` or something else more verbose?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108130



More information about the llvm-commits mailing list