Suggested by George Anzinger.

* dwarf2expr.c (execute_stack_op): Fetch the second item from the
	correct stack offset.  Include unknown opcode numbers in the error
	message.

gdb/ChangeLog

@@ -1,3 +1,10 @@
+2004-01-17  Daniel Jacobowitz  <drow@mvista.com>
+
+	Suggested by George Anzinger.
+	* dwarf2expr.c (execute_stack_op): Fetch the second item from the
+	correct stack offset.  Include unknown opcode numbers in the error
+	message.
+
 2004-01-17  Andrew Cagney  <cagney@redhat.com>
 
 	* x86-64-tdep.c (x86_64_init_abi): No need to clear

gdb/dwarf2expr.c

@@ -562,7 +562,7 @@ execute_stack_op (struct dwarf_expr_context *ctx, unsigned char *op_ptr,
 	    second = dwarf_expr_fetch (ctx, 0);
 	    dwarf_expr_pop (ctx);
 
-	    first = dwarf_expr_fetch (ctx, 1);
+	    first = dwarf_expr_fetch (ctx, 0);
 	    dwarf_expr_pop (ctx);
 
 	    val1 = value_from_longest (unsigned_address_type (), first);
@@ -660,7 +660,7 @@ execute_stack_op (struct dwarf_expr_context *ctx, unsigned char *op_ptr,
 	  goto no_push;
 
 	default:
-	  error ("Unhandled dwarf expression opcode");
+	  error ("Unhandled dwarf expression opcode 0x%x", op);
 	}
 
       /* Most things push a result value.  */