Remove TODOs for things that have been done

Also fix comment as OP_GC_BIF3 is covered by tests

Signed-off-by: Paul Guyot <pguyot@kallisys.net>

Author: pguyot

src/libAtomVM/opcodesswitch.h

@@ -2605,7 +2605,6 @@ HOT_FUNC int scheduler_entry_point(GlobalContext *glb)
                 break;
             }
 
-            //TODO: implement wait/1
             case OP_WAIT: {
                 uint32_t label;
                 DECODE_LABEL(label, pc)
@@ -2622,7 +2621,6 @@ HOT_FUNC int scheduler_entry_point(GlobalContext *glb)
                 break;
             }
 
-            //TODO: implement wait_timeout/2
             case OP_WAIT_TIMEOUT: {
                 #ifdef IMPL_EXECUTE_LOOP
                     // PC for wait_timeout_trap_handler, just before label
@@ -2936,7 +2934,6 @@ HOT_FUNC int scheduler_entry_point(GlobalContext *glb)
                 #ifdef IMPL_EXECUTE_LOOP
                     TRACE("is_number/2, label=%i, arg1=%lx\n", label, arg1);
 
-                    //TODO: check for floats too
                     if (!term_is_number(arg1)) {
                         pc = mod->labels[label];
                     }
@@ -3278,7 +3275,6 @@ HOT_FUNC int scheduler_entry_point(GlobalContext *glb)
                         #endif
 
                         #ifdef IMPL_EXECUTE_LOOP
-                            //TODO: check if src_value is a tuple
                             if (!jump_to_address && ((uint32_t) arity == cmp_value)) {
                                 jump_to_address = mod->labels[jmp_label];
                             }
@@ -3707,7 +3703,6 @@ HOT_FUNC int scheduler_entry_point(GlobalContext *glb)
 
                 #ifdef IMPL_EXECUTE_LOOP
                     term catch_term = term_from_catch_label(mod->module_index, label);
-                    //TODO: here just write to y registers is enough
                     WRITE_REGISTER(dreg, catch_term);
                 #endif
                 break;
@@ -3720,7 +3715,6 @@ HOT_FUNC int scheduler_entry_point(GlobalContext *glb)
                 TRACE("try_end/1, reg=%c%i\n", T_DEST_REG(dreg));
 
                 #ifdef IMPL_EXECUTE_LOOP
-                    //TODO: here just write to y registers is enough
                     WRITE_REGISTER(dreg, term_nil());
                 #endif
                 break;
@@ -3801,7 +3795,6 @@ HOT_FUNC int scheduler_entry_point(GlobalContext *glb)
 
                 #ifdef IMPL_EXECUTE_LOOP
                     term catch_term = term_from_catch_label(mod->module_index, label);
-                    // TODO: here just write to y registers is enough
                     WRITE_REGISTER(dreg, catch_term);
                 #endif
                 break;
@@ -3814,7 +3807,6 @@ HOT_FUNC int scheduler_entry_point(GlobalContext *glb)
                 TRACE("catch_end/1, reg=%c%i\n", T_DEST_REG(dreg));
 
                 #ifdef IMPL_EXECUTE_LOOP
-                    // TODO: here just write to y registers is enough
                     WRITE_REGISTER(dreg, term_nil());
                     // C.f. https://www.erlang.org/doc/reference_manual/expressions.html#catch-and-throw
                     switch (term_to_atom_index(x_regs[0])) {
@@ -5556,7 +5548,6 @@ HOT_FUNC int scheduler_entry_point(GlobalContext *glb)
                 break;
             }
 
-            //TODO: stub, always false
             case OP_IS_BITSTR: {
                 uint32_t label;
                 DECODE_LABEL(label, pc)
@@ -5579,8 +5570,6 @@ HOT_FUNC int scheduler_entry_point(GlobalContext *glb)
                 break;
             }
 
-            // TODO: This opcode is currently uncovered by tests.
-            // We need to implement GC bifs with arity 3, e.g. binary_part/3.
             case OP_GC_BIF3: {
                 uint32_t fail_label;
                 DECODE_LABEL(fail_label, pc);
@@ -5654,7 +5643,6 @@ HOT_FUNC int scheduler_entry_point(GlobalContext *glb)
             }
 
 #if MINIMUM_OTP_COMPILER_VERSION <= 23
-            //TODO: stub, implement recv_mark/1
             //it looks like it can be safely left unimplemented
             case OP_RECV_MARK: {
                 uint32_t label;
@@ -5665,7 +5653,6 @@ HOT_FUNC int scheduler_entry_point(GlobalContext *glb)
                 break;
             }
 
-            //TODO: stub, implement recv_set/1
             //it looks like it can be safely left unimplemented
             case OP_RECV_SET: {
                 uint32_t label;