diff options
author | Andrzej Warzynski <andrzej.warzynski@arm.com> | 2021-02-11 11:37:24 +0000 |
---|---|---|
committer | Andrzej Warzynski <andrzej.warzynski@arm.com> | 2021-02-11 11:42:57 +0000 |
commit | 0feff71eabd39fc93f3d9ec30ed16e26da6cf883 (patch) | |
tree | 7fcdd502801ce345bcdd20f077e2c76fcd5278c4 /flang/lib/Frontend/CompilerInvocation.cpp | |
parent | c16f776028ddd39a4d5ea39d5c3831b1bcbb8c02 (diff) | |
download | llvm-0feff71eabd39fc93f3d9ec30ed16e26da6cf883.zip llvm-0feff71eabd39fc93f3d9ec30ed16e26da6cf883.tar.gz llvm-0feff71eabd39fc93f3d9ec30ed16e26da6cf883.tar.bz2 |
[flang][driver] Move standard macro predefs to a dedicated method (nfc)
This patch just addresses one of the outstanding TODOs. More
specifically, it moves all the outstanding standard macro predefinitions
from `SetDefaultFortranOpts` to `setDefaultPredefinitions`. This
dedicated method for standard macro predefs was introduced in:
* https://reviews.llvm.org/D96032
Diffstat (limited to 'flang/lib/Frontend/CompilerInvocation.cpp')
-rw-r--r-- | flang/lib/Frontend/CompilerInvocation.cpp | 17 |
1 files changed, 8 insertions, 9 deletions
diff --git a/flang/lib/Frontend/CompilerInvocation.cpp b/flang/lib/Frontend/CompilerInvocation.cpp index 078cb95..23cbc21 100644 --- a/flang/lib/Frontend/CompilerInvocation.cpp +++ b/flang/lib/Frontend/CompilerInvocation.cpp @@ -328,12 +328,16 @@ void CompilerInvocation::SetDefaultFortranOpts() { std::vector<std::string> searchDirectories{"."s}; fortranOptions.searchDirectories = searchDirectories; fortranOptions.isFixedForm = false; +} + +// TODO: When expanding this method, consider creating a dedicated API for +// this. Also at some point we will need to differentiate between different +// targets and add dedicated predefines for each. +void CompilerInvocation::setDefaultPredefinitions() { + auto &fortranOptions = fortranOpts(); + const auto &frontendOptions = frontendOpts(); // Populate the macro list with version numbers and other predefinitions. - // TODO: When expanding this list of standard predefinitions, consider - // creating a dedicated API for this. Also at some point we will need to - // differentiate between different targets. - // TODO: Move to setDefaultPredefinitions fortranOptions.predefinitions.emplace_back("__flang__", "1"); fortranOptions.predefinitions.emplace_back( "__flang_major__", FLANG_VERSION_MAJOR_STRING); @@ -341,11 +345,6 @@ void CompilerInvocation::SetDefaultFortranOpts() { "__flang_minor__", FLANG_VERSION_MINOR_STRING); fortranOptions.predefinitions.emplace_back( "__flang_patchlevel__", FLANG_VERSION_PATCHLEVEL_STRING); -} - -void CompilerInvocation::setDefaultPredefinitions() { - auto &fortranOptions = fortranOpts(); - const auto &frontendOptions = frontendOpts(); // Add predefinitions based on extensions enabled if (frontendOptions.features_.IsEnabled( |