We recently changed Visit to call enter_node before enter_scope (#4203).

This change is mostly also reflected in SemanticBuilder's Visit method overrides, but a few still have enter_scope before enter_node. These are:

• visit_for_statement
• visit_for_in_statement
• visit_for_of_statement

We should alter them to align with Visit's default behavior, to avoid surprise.