Hi,

Here is a fix on top of the bitpos fixes based on the warnings
generated from gcc -Wconversion.  I have also attached the report for
review; I have not rebased since the last submission to ensure that
the line numbers don't go awry. Most of the extra warnings were either
unrelated or were the length parameter to
(store|extract)_(un)?signed_integer functions that are safe.

I have also verified that this does not cause any regressions in the
testsuite and that the gcc warnings generated after this were safe.

Regards,
Siddhesh

gdb/ChangeLog:

	* ada-lang.c (ada_lookup_struct_elt_type): Expand parameter
	DISPP to point to LONGEST.  Expand DISP to LONGEST.
	* breakpoint.c (insert_watchpoint): Expand LENGTH to LONGEST.
	(remove_watchpoint): Likewise.
	(resources_needed_watchpoint): Likewise.
	* hppa-tdep.c (hppa64_push_dummy_call): Expand OFFSET, REGNUM to
	LONGEST.
	* m32r-tdep.c (m32r_push_dummy_call): Expand STACK_ALLOC to
	LONGEST.
	* rs6000-aix-tdep.c (rs6000_push_dummy_call): Expand II to
	LONGEST.
	* tic6x-tdep.c (tic6x_push_dummy_call): Expand STACK_OFFSET to
	LONGEST.
	* valops.c (value_slice): Expand parameters LOWBOUND, LENGTH to
	LONGEST.
	* value.h (value_slice): Likewise.