[flang][nfc] Fix variable names in `FrontendOptions` & `PreprocessorOptions`
authorAndrzej Warzynski <andrzej.warzynski@arm.com>
Thu, 29 Jul 2021 12:39:10 +0000 (12:39 +0000)
committerAndrzej Warzynski <andrzej.warzynski@arm.com>
Mon, 2 Aug 2021 10:08:31 +0000 (10:08 +0000)
commit23d4c4f3fb12e127a5f07a7082f33b02082bb3f5
treec1633749019b5b68104ca58ca22a3c121622b101
parent28293918409dd3a5aa1aefa3342e50e099814cab
[flang][nfc] Fix variable names in `FrontendOptions` & `PreprocessorOptions`

As all member variables in `FrontendOptions` and `PreprocessorOptions`
are public, we should be naming them as `variable` rather than
`variable_` [1]. This patch fixes that.

Also, `FrontendOptions` & `PreprocessorOptions` are re-defined as a
structs rather than classes (all fields are meant to be public).

[1]
https://github.com/llvm/llvm-project/blob/main/flang/docs/C%2B%2Bstyle.md#naming

Differential Revision: https://reviews.llvm.org/D107062
flang/docs/FlangDriver.md
flang/include/flang/Frontend/CompilerInstance.h
flang/include/flang/Frontend/FrontendOptions.h
flang/include/flang/Frontend/PreprocessorOptions.h
flang/lib/Frontend/CompilerInstance.cpp
flang/lib/Frontend/CompilerInvocation.cpp
flang/lib/Frontend/FrontendAction.cpp
flang/lib/Frontend/FrontendActions.cpp
flang/lib/FrontendTool/ExecuteCompilerInvocation.cpp
flang/unittests/Frontend/FrontendActionTest.cpp