optimization/8848: g++ 3.3: bad code when optimizing (lifetime analysis pass?)

[snyder@fnal.gov]

>Number:         8848
>Category:       optimization
>Synopsis:       g++ 3.3: bad code when optimizing (lifetime analysis pass?)
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    unassigned
>State:          open
>Class:          wrong-code
>Submitter-Id:   net
>Arrival-Date:   Fri Dec 06 12:56:01 PST 2002
>Closed-Date:
>Last-Modified:
>Originator:     scott snyder
>Release:        3.3 20021203 (experimental)
>Organization:
>Environment:
System: Linux karma 2.4.19-emp_2419p5a829i #1 Tue Sep 3 17:42:17 EST 2002 i686 unknown
Architecture: i686

	
host: i686-pc-linux-gnu
build: i686-pc-linux-gnu
target: i686-pc-linux-gnu
configured with: ../gcc/configure --prefix=/usr/local/gcc --enable-threads=posix --enable-long-long --enable-languages=c,c++,f77
>Description:

The following program is miscompiled in linux/x86 when the
`-march=pentium2 -O1' flags are used.

I expect this program to print `3 3'; here's what it actually does:

$ g++ -o x -march=pentium2 -O1 x.cc
$ ./x
0 3
$

Here's the relevant portion of the generated code:

	movl	8(%ebp), %ecx  ; status -> ecx
	cmpl	$1, %ecx
	sete	%al
	movzbl	%al, %edx
	cmpl	$4, %ecx
	movl	$3, %eax
	cmove	%eax, %edx
	movl	$4, %eax
	cmove	%eax, %edx
	movl	%ecx, 8(%esp)   ; status
	movl	%edx, 4(%esp)   ; s
	movl	$.LC0, (%esp)
	call	printf

It looks like the comparison against `3' has been dropped.

Looking at the RTL dumps, it looks like things are going
bad during the lifetime analysis pass.

In the `bp' dump (whatever that is --- it would be incredibly useful
if the `passes' section of the documentation were kept up to date),
i see comparisons against 1, 3, and 4.  But in the `life' dump,
i see comparisions against 1, 4, and 4:

(insn 3 56 4 0 (nil) (set (reg/v:SI 58)
        (mem/f:SI (reg/f:SI 16 argp) [0 status+0 S4 A32])) 38 {*movsi_1} (nil)
    (expr_list:REG_EQUIV (mem/f:SI (reg/f:SI 16 argp) [0 status+0 S4 A32])
        (nil)))

(note 4 3 64 0 NOTE_INSN_FUNCTION_BEG)

(insn 64 4 65 0 0x400439cc (set (reg:CCZ 17 flags)
        (compare:CCZ (reg/v:SI 58)
            (const_int 1 [0x1]))) -1 (insn_list 3 (nil))
    (nil))

(insn 65 64 67 0 0x400439cc (set (reg:QI 60)
        (eq:QI (reg:CCZ 17 flags)
            (const_int 0 [0x0]))) -1 (insn_list 64 (nil))
    (expr_list:REG_DEAD (reg:CCZ 17 flags)
        (nil)))

(insn 67 65 69 0 0x400439cc (parallel [
            (set (reg:SI 61)
                (zero_extend:SI (reg:QI 60)))
            (clobber (reg:CC 17 flags))
        ]) 84 {*zero_extendqisi2_movzbw_and} (insn_list 65 (nil))
    (expr_list:REG_DEAD (reg:QI 60)
        (expr_list:REG_UNUSED (reg:CC 17 flags)
            (nil))))

(insn 69 67 37 0 0x400439cc (set (reg:SI 62)
        (const_int 3 [0x3])) -1 (nil)
    (expr_list:REG_EQUAL (const_int 3 [0x3])
        (nil)))

(insn 37 69 71 0 0x400439cc (set (reg:CCZ 17 flags)
        (compare:CCZ (reg/v:SI 58)
            (const_int 4 [0x4]))) 2 {*cmpsi_1_insn} (nil)
    (nil))

(insn 71 37 73 0 0x400439cc (set (reg:SI 63)
        (if_then_else:SI (ne (reg:CCZ 17 flags)
                (const_int 0 [0x0]))
            (reg:SI 61)
            (reg:SI 62))) 462 {*movsicc_noc} (insn_list 67 (insn_list 69 (insn_list 37 (nil))))
    (expr_list:REG_DEAD (reg:CCZ 17 flags)
        (expr_list:REG_DEAD (reg:SI 61)
            (expr_list:REG_DEAD (reg:SI 62)
                (nil)))))

(insn 73 71 72 0 0x400439cc (set (reg:SI 64)
        (const_int 4 [0x4])) -1 (nil)
    (expr_list:REG_EQUAL (const_int 4 [0x4])
        (nil)))

(insn 72 73 74 0 0x400439cc (set (reg:CCZ 17 flags)
        (compare:CCZ (reg/v:SI 58)
            (const_int 4 [0x4]))) -1 (nil)
    (nil))

(insn 74 72 48 0 0x400439cc (set (reg/v:SI 59)
        (if_then_else:SI (ne (reg:CCZ 17 flags)
                (const_int 0 [0x0]))
            (reg:SI 63)
            (reg:SI 64))) -1 (insn_list 71 (insn_list 73 (insn_list 72 (nil))))
    (expr_list:REG_DEAD (reg:CCZ 17 flags)
        (expr_list:REG_DEAD (reg:SI 63)
            (expr_list:REG_DEAD (reg:SI 64)
                (nil)))))

(insn 48 74 49 0 0x400439cc (set (mem/f:SI (plus:SI (reg/f:SI 7 esp)
                (const_int 8 [0x8])) [0 S4 A32])
        (reg/v:SI 58)) 38 {*movsi_1} (nil)
    (expr_list:REG_DEAD (reg/v:SI 58)
        (nil)))

(insn 49 48 50 0 0x400439cc (set (mem/f:SI (plus:SI (reg/f:SI 7 esp)
                (const_int 4 [0x4])) [0 S4 A32])
        (reg/v:SI 59)) 38 {*movsi_1} (insn_list 74 (nil))
    (expr_list:REG_DEAD (reg/v:SI 59)
        (nil)))

(insn 50 49 51 0 0x400439cc (set (mem/f:SI (reg/f:SI 7 esp) [0 S4 A32])
        (symbol_ref/f:SI ("*.LC0"))) 38 {*movsi_1} (nil)
    (nil))




>How-To-Repeat:

Compile this with with `-march=pentium2 -O1':

--------------------------------
extern "C" int printf (const char*, ...);

void set_parameters_nocache(int status) {
  int s = 0;
  if( status == 1 ) s=1;
  if( status == 3 ) s=3;
  if( status == 4 ) s=4;

  printf ("%d %d\n", s, status);
}



int main( ) {
  set_parameters_nocache (3);
}


--------------------------------

>Fix:
	
>Release-Note:
>Audit-Trail:
>Unformatted: