GitLab

I was tracing problem in print.c when I came across the follow code in PRTassign:

instr = ASSIGN_STMT (arg_node);
DBUG_ASSERT (instr != NULL, "instruction of N_assign is NULL");

trav_instr = TRUE;
if (NODE_TYPE (instr) == N_annotate) {
    if (global.compiler_phase < PH_cg) {
        trav_instr = FALSE;
        trav_instr = TRUE;
    }
    DBUG_EXECUTE_TAG ("PRINT_PROFILE", trav_instr = TRUE);
}

I am assuming that when we encounter an N_annotate within the ASSIGN_STMT, we should not traverse through it and print it. Yet here we are overriding the trav_instr to be always true, meaning we do. This change was added in commit a0d6a84f51.

Is this intentional?