switch to ++iValue, drop void cast & braces

Author: Andres-Salamanca

clang/lib/CIR/Dialect/IR/CIRDialect.cpp

@@ -976,9 +976,8 @@ void cir::SwitchFlatOp::build(OpBuilder &builder, OperationState &result,
                               ArrayRef<ValueRange> caseOperands) {
 
   std::vector<mlir::Attribute> caseValuesAttrs;
-  for (const APInt &val : caseValues) {
+  for (const APInt &val : caseValues)
     caseValuesAttrs.push_back(cir::IntAttr::get(value.getType(), val));
-  }
   mlir::ArrayAttr attrs = ArrayAttr::get(builder.getContext(), caseValuesAttrs);
 
   build(builder, result, value, defaultOperands, caseOperands, attrs,

clang/lib/CIR/Dialect/Transforms/FlattenCFG.cpp

@@ -374,8 +374,7 @@ class CIRSwitchOpFlattening : public mlir::OpRewritePattern<cir::SwitchOp> {
       constexpr int kSmallRangeThreshold = 64;
       if ((upperBound - lowerBound)
               .ult(llvm::APInt(32, kSmallRangeThreshold))) {
-        for (APInt iValue = lowerBound; iValue.sle(upperBound);
-             (void)iValue++) {
+        for (APInt iValue = lowerBound; iValue.sle(upperBound); ++iValue) {
           caseValues.push_back(iValue);
           caseOperands.push_back(operand);
           caseDestinations.push_back(destination);