Apply suggestions from code review

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
vgvassilev · Feb 12, 2024 · 41f91c0 · 41f91c0
1 parent c0a8b55
commit 41f91c0
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/lib/Differentiator/CladUtils.cpp b/lib/Differentiator/CladUtils.cpp
@@ -634,7 +634,7 @@ namespace clad {
     }
 
     void SetSwitchCaseSubStmt(SwitchCase* SC, Stmt* subStmt) {
-      if (auto caseStmt = dyn_cast<CaseStmt>(SC))
+      if (auto *caseStmt = dyn_cast<CaseStmt>(SC))
         caseStmt->setSubStmt(subStmt);
       else
         cast<DefaultStmt>(SC)->setSubStmt(subStmt);

diff --git a/lib/Differentiator/ReverseModeVisitor.cpp b/lib/Differentiator/ReverseModeVisitor.cpp
@@ -3296,10 +3296,10 @@ Expr* getArraySizeExpr(const ArrayType* AT, ASTContext& context,
       condExpr = GlobalStoreAndRef(condDiff.getExpr(), "_cond").getExpr();
     }
 
-    auto activeBreakContHandler = PushBreakContStmtHandler(
+    auto *activeBreakContHandler = PushBreakContStmtHandler(
         /*forSwitchStmt=*/true);
     activeBreakContHandler->BeginCFSwitchStmtScope();
-    auto SSData = PushSwitchStmtInfo();
+    auto *SSData = PushSwitchStmtInfo();
 
     if (isInsideLoop)
       SSData->switchStmtCond = condTape->Last();
@@ -3350,8 +3350,8 @@ Expr* getArraySizeExpr(const ArrayType* AT, ASTContext& context,
     // ```
     if (SSData->defaultIfBreakExpr) {
       Expr* breakCond = nullptr;
-      for (auto SC : SSData->cases) {
-        if (auto CS = dyn_cast<CaseStmt>(SC)) {
+      for (auto *SC : SSData->cases) {
+        if (auto *CS = dyn_cast<CaseStmt>(SC)) {
           if (breakCond) {
             breakCond = BuildOp(BinaryOperatorKind::BO_LAnd, breakCond,
                                 BuildOp(BinaryOperatorKind::BO_NE,
@@ -3407,7 +3407,7 @@ Expr* getArraySizeExpr(const ArrayType* AT, ASTContext& context,
     Expr* lhsClone = (CS->getLHS() ? Clone(CS->getLHS()) : nullptr);
     Expr* rhsClone = (CS->getRHS() ? Clone(CS->getRHS()) : nullptr);
 
-    auto newSC = clad_compat::CaseStmt_Create(m_Sema.getASTContext(), lhsClone,
+    auto *newSC = clad_compat::CaseStmt_Create(m_Sema.getASTContext(), lhsClone,
                                               rhsClone, noLoc, noLoc, noLoc);
 
     Expr* ifCond = BuildOp(BinaryOperatorKind::BO_EQ, newSC->getLHS(),