Use break block instead of goto

[rfalke]

Version: ed7a3e1
Exe: https://github.com/rfalke/decompiler-subjects/blob/master/from_holdec/dmi/cfg/ia32_elf/subject.exe
Source:

int intermediate_12_loop_with_break_and_block(int arg)
{
        while (1) {
                puts("head");
                if (time(0) == arg) {
                        puts("will break");
                        break;
                }
                puts("barrier");
                if (rand() == arg) {
                        break;
                }
                puts("tail");
        }
        return 0;
}

Output:

/** address: 0x08048830 */
void intermediate_12_loop_with_break_and_block(__size32 param1)
{
    int eax;            // r24

    for(;;) {
        puts("head");
        eax = time(0);
        if (eax != param1) {
            puts("barrier");
            eax = rand();
            if (param1 == eax) {
                return;
            }
            puts("tail");
        }
        else {
            goto bb0x8048889;
        }
        return;
    }
bb0x8048889:
    puts("will break");
    return;
}

Instead of the goto should be the puts("will break"); and either a break or return.

[rfalke]

Same problem with continue blocks in intermediate_15_loop_with_continue_and_block()