fixup! Optimize equality tests with known literals

bjorng · Nov 28, 2023 · 3d579ba · 3d579ba
1 parent 36316f7
commit 3d579ba
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/erts/emulator/beam/jit/arm/instr_common.cpp b/erts/emulator/beam/jit/arm/instr_common.cpp
@@ -1481,7 +1481,6 @@ void BeamModuleAssembler::emit_i_test_arity(const ArgLabel &Fail,
     a.b_ne(resolve_beam_label(Fail, disp1MB));
 }
 
-
 /*
  * ARG1 = Fist operand
  * ARG2 = Literal list
@@ -1522,7 +1521,7 @@ void BeamGlobalAssembler::emit_is_eq_exact_list_shared() {
 
 static bool is_list_of_immediates(Eterm term) {
     while (is_list(term)) {
-        Eterm* cons = list_val(term);
+        Eterm *cons = list_val(term);
         if (!is_immed(CAR(cons))) {
             return false;
         }
@@ -1551,7 +1550,8 @@ void BeamGlobalAssembler::emit_is_eq_exact_shallow_boxed_shared() {
     a.lsr(ARG3, TMP3, imm(_HEADER_ARITY_OFFS));
     a.sub(ARG3, ARG3, imm(1));
 
-    a.bind(loop); {
+    a.bind(loop);
+    {
         a.ldp(TMP3, TMP4, arm::Mem(TMP1).post(16));
         a.ldp(TMP5, TMP6, arm::Mem(TMP2).post(16));
         a.cmp(TMP3, TMP5);
@@ -1579,7 +1579,7 @@ void BeamGlobalAssembler::emit_is_eq_exact_shallow_boxed_shared() {
 
 static bool is_shallow_boxed(Eterm term) {
     if (is_tuple(term)) {
-        Eterm* tuple_ptr = tuple_val(term);
+        Eterm *tuple_ptr = tuple_val(term);
         for (int i = 1; i <= arityval(*tuple_ptr); i++) {
             if (!is_immed(tuple_ptr[i])) {
                 return false;

diff --git a/erts/emulator/beam/jit/x86/instr_common.cpp b/erts/emulator/beam/jit/x86/instr_common.cpp
@@ -1434,7 +1434,7 @@ void BeamGlobalAssembler::emit_is_eq_exact_list_shared() {
 
 static bool is_list_of_immediates(Eterm term) {
     while (is_list(term)) {
-        Eterm* cons = list_val(term);
+        Eterm *cons = list_val(term);
         if (!is_immed(CAR(cons))) {
             return false;
         }