don't break before multi-line string. don't break before assignment operator

Author: soheilshahrouz

.clang-format

@@ -15,7 +15,7 @@ AllowShortIfStatementsOnASingleLine: true
 AllowShortLoopsOnASingleLine: false
 AlwaysBreakAfterDefinitionReturnType: None
 AlwaysBreakAfterReturnType: None
-AlwaysBreakBeforeMultilineStrings: true
+AlwaysBreakBeforeMultilineStrings: false
 AlwaysBreakTemplateDeclarations: true
 BinPackArguments: true
 BinPackParameters: false
@@ -34,7 +34,7 @@ BraceWrapping:
   SplitEmptyFunction: false
   SplitEmptyRecord: true
   SplitEmptyNamespace: true
-BreakBeforeBinaryOperators: All
+BreakBeforeBinaryOperators: NonAssignment
 BreakBeforeBraces: Custom
 BreakBeforeInheritanceComma: false
 BreakBeforeTernaryOperators: true

libs/librtlnumber/src/include/internal_bits.hpp

@@ -61,110 +61,94 @@ constexpr bit_value_t l_buf[4] = {
     _0, _1, _x, _x};
 _static_unused(l_buf)
 
-    constexpr bit_value_t l_not[4]
-    = {
+    constexpr bit_value_t l_not[4] = {
         /*   0   1   x   z 	<- a */
         _1, _0, _x, _x};
 _static_unused(l_not)
 
-    constexpr bit_value_t is_unk[4]
-    = {
+    constexpr bit_value_t is_unk[4] = {
         /*	 0   1   x   z  <- a*/
         _0, _0, _1, _1};
 _static_unused(is_unk)
 
-    constexpr bit_value_t is_x_bit[4]
-    = {
+    constexpr bit_value_t is_x_bit[4] = {
         /*	 0   1   x   z  <- a*/
         _0, _0, _1, _0};
 _static_unused(is_x_bit)
 
-    constexpr bit_value_t is_z_bit[4]
-    = {
+    constexpr bit_value_t is_z_bit[4] = {
         /*	 0   1   x   z  <- a*/
         _0, _0, _0, _1};
 _static_unused(is_z_bit)
 
-    constexpr bit_value_t is_one_bit[4]
-    = {
+    constexpr bit_value_t is_one_bit[4] = {
         /*	 0   1   x   z  <- a*/
         _0, _1, _0, _0};
 _static_unused(is_one_bit)
 
-    constexpr bit_value_t is_zero_bit[4]
-    = {
+    constexpr bit_value_t is_zero_bit[4] = {
         /*	 0   1   x   z  <- a*/
         _1, _0, _0, _0};
 _static_unused(is_zero_bit)
 
-    constexpr bit_value_t l_and[4][4]
-    = {
+    constexpr bit_value_t l_and[4][4] = {
         /* a  /	 0   1   x   z 	<-b */
         /* 0 */ {_0, _0, _0, _0},
         /* 1 */ {_0, _1, _x, _x},
         /* x */ {_0, _x, _x, _x},
         /* z */ {_0, _x, _x, _x}};
 _static_unused(l_and)
 
-    constexpr bit_value_t l_nand[4][4]
-    = unroll_2d_invert(l_and);
+    constexpr bit_value_t l_nand[4][4] = unroll_2d_invert(l_and);
 _static_unused(l_nand)
 
-    constexpr bit_value_t l_or[4][4]
-    = {
+    constexpr bit_value_t l_or[4][4] = {
         /* a  /	 0   1   x   z 	<-b */
         /* 0 */ {_0, _1, _x, _x},
         /* 1 */ {_1, _1, _1, _1},
         /* x */ {_x, _1, _x, _x},
         /* z */ {_x, _1, _x, _x}};
 _static_unused(l_or)
 
-    constexpr bit_value_t l_nor[4][4]
-    = unroll_2d_invert(l_or);
+    constexpr bit_value_t l_nor[4][4] = unroll_2d_invert(l_or);
 _static_unused(l_nor)
 
-    constexpr bit_value_t l_xor[4][4]
-    = {
+    constexpr bit_value_t l_xor[4][4] = {
         /* a  /	 0   1   x   z 	<-b */
         /* 0 */ {_0, _1, _x, _x},
         /* 1 */ {_1, _0, _x, _x},
         /* x */ {_x, _x, _x, _x},
         /* z */ {_x, _x, _x, _x}};
 _static_unused(l_xor)
 
-    constexpr bit_value_t l_xnor[4][4]
-    = unroll_2d_invert(l_xor);
+    constexpr bit_value_t l_xnor[4][4] = unroll_2d_invert(l_xor);
 _static_unused(l_xnor)
 
-    constexpr bit_value_t l_notif1[4][4]
-    = {
+    constexpr bit_value_t l_notif1[4][4] = {
         /* in /	 0   1   x   z 	<-control */
         /* 0 */ {_z, _1, _x, _x},
         /* 1 */ {_z, _0, _x, _x},
         /* x */ {_z, _x, _x, _x},
         /* z */ {_z, _x, _x, _x}};
 _static_unused(l_notif1)
 
-    constexpr bit_value_t l_notif0[4][4]
-    = {
+    constexpr bit_value_t l_notif0[4][4] = {
         /* in /	 0   1   x   z 	<-control */
         /* 0 */ {_1, _z, _x, _x},
         /* 1 */ {_0, _z, _x, _x},
         /* x */ {_x, _z, _x, _x},
         /* z */ {_x, _z, _x, _x}};
 _static_unused(l_notif0)
 
-    constexpr bit_value_t l_bufif1[4][4]
-    = {
+    constexpr bit_value_t l_bufif1[4][4] = {
         /* in /	 0   1   x   z 	<-control */
         /* 0 */ {_z, _0, _x, _x},
         /* 1 */ {_z, _1, _x, _x},
         /* x */ {_z, _x, _x, _x},
         /* z */ {_z, _x, _x, _x}};
 _static_unused(l_bufif1)
 
-    constexpr bit_value_t l_bufif0[4][4]
-    = {
+    constexpr bit_value_t l_bufif0[4][4] = {
         /* in /	 0   1   x   z 	<-control */
         /* 0 */ {_0, _z, _x, _x},
         /* 1 */ {_1, _z, _x, _x},
@@ -176,26 +160,23 @@ _static_unused(l_bufif0)
      *  Tran NO SUPPORT FOR THESE YET 
      */
     /* cmos gates */
-    constexpr bit_value_t l_rpmos[4][4]
-    = {
+    constexpr bit_value_t l_rpmos[4][4] = {
         /* in /	 0   1   x   z 	<-control */
         /* 0 */ {_0, _z, _x, _x},
         /* 1 */ {_1, _z, _x, _x},
         /* x */ {_x, _z, _x, _x},
         /* z */ {_z, _z, _z, _z}};
 _static_unused(l_rpmos)
 
-    constexpr bit_value_t l_rnmos[4][4]
-    = {
+    constexpr bit_value_t l_rnmos[4][4] = {
         /* in /	 0   1   x   z 	<-control */
         /* 0 */ {_z, _0, _x, _x},
         /* 1 */ {_z, _1, _x, _x},
         /* x */ {_z, _x, _x, _x},
         /* z */ {_z, _z, _z, _z}};
 _static_unused(l_rnmos)
 
-    constexpr bit_value_t l_nmos[4][4]
-    = {
+    constexpr bit_value_t l_nmos[4][4] = {
         /* in /	 0   1   x   z 	<-control */
         /* 0 */ {_z, _0, _x, _x},
         /* 1 */ {_z, _1, _x, _x},
@@ -204,8 +185,7 @@ _static_unused(l_rnmos)
 _static_unused(l_nmos)
 
     // see table 5-21 p:54 IEEE 1364-2005
-    constexpr bit_value_t l_ternary[4][4]
-    = {
+    constexpr bit_value_t l_ternary[4][4] = {
         /* in /	 0   1   x   z 	<-control */
         /* 0 */ {_0, _x, _x, _x},
         /* 1 */ {_x, _1, _x, _x},
@@ -217,70 +197,61 @@ _static_unused(l_ternary)
      * these extend the library and simplify the process
      */
     /* helper */
-    constexpr bit_value_t l_unk[4][4]
-    = {
+    constexpr bit_value_t l_unk[4][4] = {
         /* in /	 0   1   x   z 	<-control */
         /* 0 */ {_x, _x, _x, _x},
         /* 1 */ {_x, _x, _x, _x},
         /* x */ {_x, _x, _x, _x},
         /* z */ {_x, _x, _x, _x}};
 _static_unused(l_unk)
 
-    constexpr bit_value_t l_case_eq[4][4]
-    = {
+    constexpr bit_value_t l_case_eq[4][4] = {
         /* a  /	 0   1   x   z 	<-b */
         /* 0 */ {_1, _0, _0, _0},
         /* 1 */ {_0, _1, _0, _0},
         /* x */ {_0, _0, _1, _0},
         /* z */ {_0, _0, _0, _1}};
 _static_unused(l_case_eq)
 
-    constexpr bit_value_t l_lt[4][4]
-    = {
+    constexpr bit_value_t l_lt[4][4] = {
         /* a  /	 0   1   x   z 	<-b */
         /* 0 */ {_0, _1, _x, _x},
         /* 1 */ {_0, _0, _x, _x},
         /* x */ {_x, _x, _x, _x},
         /* z */ {_x, _x, _x, _x}};
 _static_unused(l_lt)
 
-    constexpr bit_value_t l_gt[4][4]
-    = {
+    constexpr bit_value_t l_gt[4][4] = {
         /* a  /	 0   1   x   z 	<-b */
         /* 0 */ {_0, _0, _x, _x},
         /* 1 */ {_1, _0, _x, _x},
         /* x */ {_x, _x, _x, _x},
         /* z */ {_x, _x, _x, _x}};
 _static_unused(l_gt)
 
-    constexpr bit_value_t l_eq[4][4]
-    = unroll_2d(l_xnor);
+    constexpr bit_value_t l_eq[4][4] = unroll_2d(l_xnor);
 _static_unused(l_eq)
 
-    constexpr bit_value_t l_sum[4][4][4]
-    = {
+    constexpr bit_value_t l_sum[4][4][4] = {
         /* c_in */
         /* 0 */ unroll_2d(l_xor),
         /* 1 */ unroll_2d(l_xnor),
         /* x */ unroll_2d(l_unk),
         /* z */ unroll_2d(l_unk)};
 _static_unused(l_sum)
 
-    constexpr bit_value_t l_carry[4][4][4]
-    = {
+    constexpr bit_value_t l_carry[4][4][4] = {
         /* c_in */
         /* 0 */ unroll_2d(l_and),
         /* 1 */ unroll_2d(l_or),
         /* x */ unroll_2d(l_ternary),
         /* z */ unroll_2d(l_ternary)};
 _static_unused(l_carry)
 
-    constexpr bit_value_t l_half_carry[4][4]
-    = unroll_2d(l_carry[_0]);
+    constexpr bit_value_t l_half_carry[4][4] = unroll_2d(l_carry[_0]);
 _static_unused(l_half_carry)
 
-    constexpr bit_value_t l_half_sum[4][4]
-    = unroll_2d(l_sum[_0]);
+    constexpr bit_value_t l_half_sum[4][4] = unroll_2d(l_sum[_0]);
 _static_unused(l_half_sum)
 
     static char bit_to_c(bit_value_t bit, bool uppercase) {

libs/libvtrutil/src/tl_optional.hpp

@@ -505,8 +505,7 @@ struct optional_copy_assign_base<T, false> : optional_move_base<T> {
         return *this;
     }
     optional_copy_assign_base&
-    operator=(optional_copy_assign_base&& rhs)
-        = default;
+    operator=(optional_copy_assign_base&& rhs) = default;
 };
 
 // This class manages conditionally having a trivial move assignment operator
@@ -534,8 +533,7 @@ struct optional_move_assign_base<T, false> : optional_copy_assign_base<T> {
     optional_move_assign_base(optional_move_assign_base&& rhs) = default;
 
     optional_move_assign_base&
-    operator=(const optional_move_assign_base& rhs)
-        = default;
+    operator=(const optional_move_assign_base& rhs) = default;
 
     optional_move_assign_base&
     operator=(optional_move_assign_base&& rhs) noexcept(
@@ -553,8 +551,7 @@ struct optional_delete_ctor_base {
     optional_delete_ctor_base(const optional_delete_ctor_base&) = default;
     optional_delete_ctor_base(optional_delete_ctor_base&&) noexcept = default;
     optional_delete_ctor_base&
-    operator=(const optional_delete_ctor_base&)
-        = default;
+    operator=(const optional_delete_ctor_base&) = default;
     optional_delete_ctor_base&
     operator=(optional_delete_ctor_base&&) noexcept = default;
 };
@@ -565,8 +562,7 @@ struct optional_delete_ctor_base<T, true, false> {
     optional_delete_ctor_base(const optional_delete_ctor_base&) = default;
     optional_delete_ctor_base(optional_delete_ctor_base&&) noexcept = delete;
     optional_delete_ctor_base&
-    operator=(const optional_delete_ctor_base&)
-        = default;
+    operator=(const optional_delete_ctor_base&) = default;
     optional_delete_ctor_base&
     operator=(optional_delete_ctor_base&&) noexcept = default;
 };
@@ -577,8 +573,7 @@ struct optional_delete_ctor_base<T, false, true> {
     optional_delete_ctor_base(const optional_delete_ctor_base&) = delete;
     optional_delete_ctor_base(optional_delete_ctor_base&&) noexcept = default;
     optional_delete_ctor_base&
-    operator=(const optional_delete_ctor_base&)
-        = default;
+    operator=(const optional_delete_ctor_base&) = default;
     optional_delete_ctor_base&
     operator=(optional_delete_ctor_base&&) noexcept = default;
 };
@@ -589,8 +584,7 @@ struct optional_delete_ctor_base<T, false, false> {
     optional_delete_ctor_base(const optional_delete_ctor_base&) = delete;
     optional_delete_ctor_base(optional_delete_ctor_base&&) noexcept = delete;
     optional_delete_ctor_base&
-    operator=(const optional_delete_ctor_base&)
-        = default;
+    operator=(const optional_delete_ctor_base&) = default;
     optional_delete_ctor_base&
     operator=(optional_delete_ctor_base&&) noexcept = default;
 };
@@ -605,8 +599,7 @@ struct optional_delete_assign_base {
     optional_delete_assign_base(const optional_delete_assign_base&) = default;
     optional_delete_assign_base(optional_delete_assign_base&&) noexcept = default;
     optional_delete_assign_base&
-    operator=(const optional_delete_assign_base&)
-        = default;
+    operator=(const optional_delete_assign_base&) = default;
     optional_delete_assign_base&
     operator=(optional_delete_assign_base&&) noexcept = default;
 };
@@ -617,8 +610,7 @@ struct optional_delete_assign_base<T, true, false> {
     optional_delete_assign_base(const optional_delete_assign_base&) = default;
     optional_delete_assign_base(optional_delete_assign_base&&) noexcept = default;
     optional_delete_assign_base&
-    operator=(const optional_delete_assign_base&)
-        = default;
+    operator=(const optional_delete_assign_base&) = default;
     optional_delete_assign_base&
     operator=(optional_delete_assign_base&&) noexcept = delete;
 };
@@ -629,8 +621,7 @@ struct optional_delete_assign_base<T, false, true> {
     optional_delete_assign_base(const optional_delete_assign_base&) = default;
     optional_delete_assign_base(optional_delete_assign_base&&) noexcept = default;
     optional_delete_assign_base&
-    operator=(const optional_delete_assign_base&)
-        = delete;
+    operator=(const optional_delete_assign_base&) = delete;
     optional_delete_assign_base&
     operator=(optional_delete_assign_base&&) noexcept = default;
 };
@@ -641,8 +632,7 @@ struct optional_delete_assign_base<T, false, false> {
     optional_delete_assign_base(const optional_delete_assign_base&) = default;
     optional_delete_assign_base(optional_delete_assign_base&&) noexcept = default;
     optional_delete_assign_base&
-    operator=(const optional_delete_assign_base&)
-        = delete;
+    operator=(const optional_delete_assign_base&) = delete;
     optional_delete_assign_base&
     operator=(optional_delete_assign_base&&) noexcept = delete;
 };

libs/libvtrutil/src/vtr_prefix_sum.h

@@ -219,7 +219,7 @@ class PrefixSum2D {
                                     prefix_sum_[x - 1][y - 1];
             }
         }
-    } 
+    }
 
     /**
      * @brief Constructs a 2D prefix sum from a 2D grid of values.

odin_ii/src/ast/ast_util.cpp

@@ -113,8 +113,7 @@ void add_top_module_to_ast(ast_t* ast, ast_node_t* to_add) {
     oassert(ast);
     if (to_add) {
         ast->top_modules_count += 1;
-        ast->top_modules
-            = (ast_node_t**)vtr::realloc(ast->top_modules, sizeof(ast_node_t*) * ast->top_modules_count);
+        ast->top_modules = (ast_node_t**)vtr::realloc(ast->top_modules, sizeof(ast_node_t*) * ast->top_modules_count);
 
         ast->top_modules[ast->top_modules_count - 1] = to_add;
     }