Skip to content

Commit 89053e4

Browse files
committed
[flang][driver] Partial revert of D157837
This is a partial revert of https://reviews.llvm.org/D157837 - it turns out that the LLVM test suite needs to be updated first not to use any of the unsupported Flang options: * https://github.com/llvm/llvm-test-suite Sample errors: ``` flang-new: error: unknown argument: '-fbounds-check' flang-new: error: unknown argument: '-fcheck=all' flang-new: error: unknown argument: '-fcheck-array-temporaries' ``` Once the test suite is updated, we can restore the reverted setting. Broken bot: * https://lab.llvm.org/buildbot/#/builders/197/builds/9001 Differential Revision: https://reviews.llvm.org/D158289
1 parent d6e7c16 commit 89053e4

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

clang/lib/Driver/Driver.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6494,8 +6494,8 @@ Driver::getOptionVisibilityMask(bool UseDriverMode) const {
64946494
return llvm::opt::Visibility(options::CLOption);
64956495
if (IsDXCMode())
64966496
return llvm::opt::Visibility(options::DXCOption);
6497-
if (IsFlangMode()) {
6498-
return llvm::opt::Visibility(options::FlangOption);
6497+
if (IsFlangMode()) {
6498+
return llvm::opt::Visibility(options::ClangOption | options::FlangOption);
64996499
}
65006500
return llvm::opt::Visibility(options::ClangOption);
65016501
}

0 commit comments

Comments
 (0)