Skip to content

Commit b10899d

Browse files
committed
[clang] Fix -Wlogical-op-parentheses in Clang.cpp (NFC)
/Users/jiefu/llvm-project/clang/lib/Driver/ToolChains/Clang.cpp:7345:56: error: '&&' within '||' [-Werror,-Wlogical-op-parentheses] (WholeProgramVTables || SanitizeArgs.needsLTO()) && ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~ /Users/jiefu/llvm-project/clang/lib/Driver/ToolChains/Clang.cpp:7345:56: note: place parentheses around the '&&' expression to silence this warning (WholeProgramVTables || SanitizeArgs.needsLTO()) && ^ ( 1 error generated.
1 parent fd2254b commit b10899d

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

clang/lib/Driver/ToolChains/Clang.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7342,8 +7342,8 @@ void Clang::ConstructJob(Compilation &C, const JobAction &JA,
73427342
}
73437343

73447344
bool DefaultsSplitLTOUnit =
7345-
(WholeProgramVTables || SanitizeArgs.needsLTO()) &&
7346-
(LTOMode == LTOK_Full || TC.canSplitThinLTOUnit()) ||
7345+
((WholeProgramVTables || SanitizeArgs.needsLTO()) &&
7346+
(LTOMode == LTOK_Full || TC.canSplitThinLTOUnit())) ||
73477347
(!Triple.isPS4() && UnifiedLTO);
73487348
bool SplitLTOUnit =
73497349
Args.hasFlag(options::OPT_fsplit_lto_unit,

0 commit comments

Comments
 (0)