Add braces around if/for bodies

Author: kparzysz

flang/include/flang/Semantics/dump-expr.h

@@ -67,18 +67,21 @@ class DumpEvaluateExpr {
   }
   template <typename A> void Show(const std::vector<A> &x) {
     Indent("vector");
-    for (const auto &v : x)
+    for (const auto &v : x) {
       Show(v);
+    }
     Outdent();
   }
   void Show(const evaluate::BOZLiteralConstant &);
   void Show(const evaluate::NullPointer &);
   template <typename T> void Show(const evaluate::Constant<T> &x) {
     if constexpr (T::category == common::TypeCategory::Derived) {
       Indent("derived constant");
-      for (const auto &map : x.values())
-        for (const auto &pair : map)
+      for (const auto &map : x.values()) {
+        for (const auto &pair : map) {
           Show(pair.second.value());
+        }
+      }
       Outdent();
     } else {
       Print("constant");
@@ -130,8 +133,9 @@ class DumpEvaluateExpr {
   template <typename T>
   void Show(const evaluate::ArrayConstructorValues<T> &x) {
     Indent("array constructor value");
-    for (auto &v : x)
+    for (auto &v : x) {
       Show(v);
+    }
     Outdent();
   }
   template <typename T> void Show(const evaluate::ImpliedDo<T> &x) {

flang/lib/Semantics/dump-expr.cpp

@@ -69,10 +69,11 @@ void DumpEvaluateExpr::Show(const evaluate::Component &x) {
 
 void DumpEvaluateExpr::Show(const evaluate::NamedEntity &x) {
   Indent("named entity");
-  if (const auto *component{x.UnwrapComponent()})
+  if (const auto *component{x.UnwrapComponent()}) {
     Show(*component);
-  else
+  } else {
     Show(x.GetFirstSymbol());
+  }
   Outdent();
 }
 
@@ -130,8 +131,9 @@ void DumpEvaluateExpr::Show(
 
 void DumpEvaluateExpr::Show(const DerivedTypeSpec &x) {
   Indent("derived type spec");
-  for (auto &v : x.parameters())
+  for (auto &v : x.parameters()) {
     Show(v);
+  }
   Outdent();
 }
 
@@ -143,8 +145,9 @@ void DumpEvaluateExpr::Show(
 void DumpEvaluateExpr::Show(const evaluate::StructureConstructor &x) {
   Indent("structure constructor");
   Show(x.derivedTypeSpec());
-  for (auto &v : x)
+  for (auto &v : x) {
     Show(v);
+  }
   Outdent();
 }
 
@@ -162,21 +165,23 @@ void DumpEvaluateExpr::Show(const evaluate::ComplexPart &x) {
 
 void DumpEvaluateExpr::Show(const evaluate::ActualArgument &x) {
   Indent("actual argument");
-  if (const auto *symbol{x.GetAssumedTypeDummy()})
+  if (const auto *symbol{x.GetAssumedTypeDummy()}) {
     Show(*symbol);
-  else
+  } else {
     Show(x.UnwrapExpr());
+  }
   Outdent();
 }
 
 void DumpEvaluateExpr::Show(const evaluate::ProcedureDesignator &x) {
   Indent("procedure designator");
-  if (const auto *component{x.GetComponent()})
+  if (const auto *component{x.GetComponent()}) {
     Show(*component);
-  else if (const auto *symbol{x.GetSymbol()})
+  } else if (const auto *symbol{x.GetSymbol()}) {
     Show(*symbol);
-  else
+  } else {
     Show(DEREF(x.GetSpecificIntrinsic()));
+  }
   Outdent();
 }
 
@@ -200,8 +205,9 @@ void DumpEvaluateExpr::Indent(llvm::StringRef s) {
 }
 
 void DumpEvaluateExpr::Outdent() {
-  if (level_)
+  if (level_) {
     level_--;
+  }
   Print("}");
 }