[clang-tools-extra] [clang-tidy][NFC] Fix readability-redundant-parentheses warnings (PR #182766)

[via cfe-commits]

llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang-tidy

Author: Baranov Victor (vbvictor)

<details>
<summary>Changes</summary>



---
Full diff: https://github.com/llvm/llvm-project/pull/182766.diff


2 Files Affected:

- (modified) clang-tools-extra/clang-tidy/altera/UnrollLoopsCheck.cpp (+4-4) 
- (modified) clang-tools-extra/clang-tidy/bugprone/NondeterministicPointerIterationOrderCheck.cpp (+2-2) 


``````````diff
diff --git a/clang-tools-extra/clang-tidy/altera/UnrollLoopsCheck.cpp b/clang-tools-extra/clang-tidy/altera/UnrollLoopsCheck.cpp
index c759e5b5ee7bd..15fa75f19ef28 100644
--- a/clang-tools-extra/clang-tidy/altera/UnrollLoopsCheck.cpp
+++ b/clang-tools-extra/clang-tidy/altera/UnrollLoopsCheck.cpp
@@ -207,20 +207,20 @@ bool UnrollLoopsCheck::hasLargeNumIterations(const Stmt *Statement,
     if (!extractValue(ConstantValue, Op, Context))
       return true;
     switch (Op->getOpcode()) {
-    case (BO_AddAssign):
+    case BO_AddAssign:
       Iterations =
           std::ceil(static_cast<float>(EndValue - InitValue) / ConstantValue);
       break;
-    case (BO_SubAssign):
+    case BO_SubAssign:
       Iterations =
           std::ceil(static_cast<float>(InitValue - EndValue) / ConstantValue);
       break;
-    case (BO_MulAssign):
+    case BO_MulAssign:
       Iterations = 1 + ((std::log(static_cast<double>(EndValue)) -
                          std::log(static_cast<double>(InitValue))) /
                         std::log(static_cast<double>(ConstantValue)));
       break;
-    case (BO_DivAssign):
+    case BO_DivAssign:
       Iterations = 1 + ((std::log(static_cast<double>(InitValue)) -
                          std::log(static_cast<double>(EndValue))) /
                         std::log(static_cast<double>(ConstantValue)));
diff --git a/clang-tools-extra/clang-tidy/bugprone/NondeterministicPointerIterationOrderCheck.cpp b/clang-tools-extra/clang-tidy/bugprone/NondeterministicPointerIterationOrderCheck.cpp
index fdb903a333234..d7455be873496 100644
--- a/clang-tools-extra/clang-tidy/bugprone/NondeterministicPointerIterationOrderCheck.cpp
+++ b/clang-tools-extra/clang-tidy/bugprone/NondeterministicPointerIterationOrderCheck.cpp
@@ -48,7 +48,7 @@ void NondeterministicPointerIterationOrderCheck::check(
   const auto *ForRangePointers =
       Result.Nodes.getNodeAs<CXXForRangeStmt>("cxxForRangeStmt");
 
-  if ((ForRangePointers) && !(ForRangePointers->getBeginLoc().isMacroID())) {
+  if (ForRangePointers && !(ForRangePointers->getBeginLoc().isMacroID())) {
     const auto *RangeInit = Result.Nodes.getNodeAs<Stmt>("rangeinit");
     if (const auto *ClassTemplate =
             Result.Nodes.getNodeAs<ClassTemplateSpecializationDecl>(
@@ -67,7 +67,7 @@ void NondeterministicPointerIterationOrderCheck::check(
   }
   const auto *SortPointers = Result.Nodes.getNodeAs<Stmt>("sortsemantic");
 
-  if ((SortPointers) && !(SortPointers->getBeginLoc().isMacroID())) {
+  if (SortPointers && !(SortPointers->getBeginLoc().isMacroID())) {
     const SourceRange R = SortPointers->getSourceRange();
     diag(R.getBegin(), "sorting pointers is nondeterministic") << R;
   }

``````````

</details>


https://github.com/llvm/llvm-project/pull/182766