[PATCH] D107062: [flang][nfc] Fix variable names in `FrontendOptions`

[Andrzej Warzynski via Phabricator via llvm-commits]

awarzynski added inline comments.


================
Comment at: flang/include/flang/Frontend/FrontendOptions.h:253
+      : showHelp(false), showVersion(false), instrumentedParse(false),
+        needProvenanceRangeToCharBlockMappings(false) {}
 
----------------
klausler wrote:
> It is confusing to have data member initialization mixed between the constructor and the data member declarations when all of initialization expressions are constants.
I agree that the location of this constructor is unfortunate. Let me move it to the top.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107062