Skip to content

Commit 19bb9f2

Browse files
committed
Implement -Wpass-global-variable flag for flang
1 parent 0d92b7f commit 19bb9f2

File tree

3 files changed

+5
-1
lines changed

3 files changed

+5
-1
lines changed

flang/include/flang/Support/Fortran-features.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ ENUM_CLASS(UsageWarning, Portability, PointerToUndefinable,
7979
CompatibleDeclarationsFromDistinctModules,
8080
NullActualForDefaultIntentAllocatable, UseAssociationIntoSameNameSubprogram,
8181
HostAssociatedIntentOutInSpecExpr, NonVolatilePointerToVolatile,
82-
RealConstantWidening, VolatileOrAsynchronousTemporary)
82+
RealConstantWidening, VolatileOrAsynchronousTemporary, PassGlobalVariable)
8383

8484
using LanguageFeatures = EnumSet<LanguageFeature, LanguageFeature_enumSize>;
8585
using UsageWarnings = EnumSet<UsageWarning, UsageWarning_enumSize>;

flang/lib/Support/Fortran-features.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -149,6 +149,7 @@ LanguageFeatureControl::LanguageFeatureControl() {
149149
warnUsage_.set(UsageWarning::HostAssociatedIntentOutInSpecExpr);
150150
warnUsage_.set(UsageWarning::NonVolatilePointerToVolatile);
151151
warnUsage_.set(UsageWarning::RealConstantWidening);
152+
warnUsage_.set(UsageWarning::PassGlobalVariable);
152153
// New warnings, on by default
153154
warnLanguage_.set(LanguageFeature::SavedLocalInSpecExpr);
154155
warnLanguage_.set(LanguageFeature::NullActualForAllocatable);

flang/unittests/Common/FortranFeaturesTest.cpp

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -556,6 +556,9 @@ TEST(FortranFeaturesTest, CamelCaseToLowerCaseHyphenated) {
556556
EXPECT_EQ(CamelCaseToLowerCaseHyphenated(
557557
EnumToString(UsageWarning::NonVolatilePointerToVolatile)),
558558
"non-volatile-pointer-to-volatile");
559+
EXPECT_EQ(
560+
CamelCaseToLowerCaseHyphenated(EnumToString(UsageWarning::PassGlobalVariable)),
561+
"pass-global-variable");
559562
}
560563

561564
TEST(FortranFeaturesTest, HintLanguageControlFlag) {

0 commit comments

Comments
 (0)