Hi,

I have committed[1] more changes that result in using TYPE_LENGTH
directly instead of using a local variable.  In amd64-tdep and
bfin-tdep, I use this in the gdb_assert so that the LEN data type then
does not need to be changed.  Other cases are the usual elimination of
the variable to use TYPE_LENGTH directly.

This is to reduce the size of the bitpos expansion patch[2].

Regards,
Siddhesh

[1] http://sourceware.org/ml/gdb-cvs/2012-09/msg00147.html
[2] http://sourceware.org/ml/gdb-patches/2012-08/msg00144.html

        * amd64-tdep.c (amd64_return_value): Use TYPE_LENGTH directly.
        * bfin-tdep.c (bfin_extract_return_value): Likewise.
        (bfin_store_return_value): Likewise.
        * cris-tdep.c (cris_store_return_value): Likewise.
        (cris_extract_return_value): Likewise.
        * h8300-tdep.c (h8300_extract_return_value): Likewise.
        * hppa-tdep.c (hppa64_return_value): Likewise.
        * lm32-tdep.c (lm32_store_return_value): Likewise.
        * microblaze-tdep.c (microblaze_store_return_value): Likewise.
        * spu-tdep.c (spu_value_from_register): Likewise.
        * vax-tdep.c (vax_return_value): Likewise.