Regression gdb.cp/temargs.exp: test value of P in inner_m [Re: [patch] Defer symbol addition until construction is complete]

[Jan Kratochvil <jan.kratochvil@redhat.com>]

On Mon, 09 Aug 2010 20:38:26 +0200, sami wagiaalla wrote:
> Complete symbol construction before adding to pending list.
> 
> 2010-08-09  Sami Wagiaalla  <swagiaal@redhat.com>
> 
> 	* dwarf2read.c (new_symbol): Add symbol to variable list at end of
> 	function after symbol construction is complete.
> 	Do the same for template symbol addition to template_symbols list.

checked-in as:
http://sourceware.org/ml/gdb-cvs/2010-08/msg00040.html
3b26cb4b133031280fa022e2a06e58f333ed5e8d

> Is this cool ?

This patch has a cold regression. :-)

-PASS: gdb.cp/temargs.exp: test value of P in inner_m
+FAIL: gdb.cp/temargs.exp: test value of P in inner_m
:
print P == &a_global
$4 = true
(gdb) PASS: gdb.cp/temargs.exp: test value of P in inner_m
->
print P == &a_global
No symbol "P" in current context.
(gdb) FAIL: gdb.cp/temargs.exp: test value of P in inner_m

It affects {x86_64,x86_64-m32,i686}-fedora14snapshot-linux-gnu.
It does not affect {x86_64,x86_64-m32,i686}-fedora13-linux-gnu.

PASS->FAIL gcc-4.5.0-3.fc14.{x86_64,i686}
XFAIL      gcc-4.4.4-10.fc13.{x86_64,i686}
PASS->FAIL GNU C++ 4.6.0 20100810 (experimental)
PASS->FAIL GNU C++ 4.5.2 20100810 (prerelease)
XFAIL      GNU C++ 4.4.5 20100810 (prerelease)

XFAIL is:
print P == &a_global
No symbol "P" in current context.
(gdb) XFAIL: gdb.cp/temargs.exp: test value of P in inner_m


Thanks,
Jan