[PATCH] D100649: [flang][driver] Fine-tune `-fdebug-dump-symbols`

[Andrzej Warzynski via Phabricator via llvm-commits]

awarzynski added inline comments.


================
Comment at: flang/lib/Frontend/FrontendActions.cpp:284
 void DebugDumpSymbolsAction::ExecuteAction() {
+  CompilerInstance &ci = this->instance();
   auto &semantics = this->semantics();
----------------
AMDChirag wrote:
> Shouldn't the local variables be named in PascalCase?
In LLVM/Clang - yes, in MLIR/Flang -no :) It's basically lowerCase everywhere in Flang. That's not really explicitly mentioned in the [[ https://releases.llvm.org/11.0.0/tools/flang/docs/C++style.html | coding style ]], but that's the style used in this file and in other parts of Flang.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100649