recent i386-tdep change breaks build

[Robert Lipe <robertlipe at usa dot net>]

I haven't roamed the CVS logs to be sure Mark's change is the smoking
gun, but this can't be right.

void
i386_extract_return_value (struct type *type, char *regbuf, char *valbuf)
{
  int len = TYPE_LENGTH (type);

  if (TYPE_CODE (type) == TYPE_CODE_STRUCT
      && TYPE_NFIELDS (type) == 1)
    return i386_extract_return_value (TYPE_FIELD_TYPE (type, 0),
                                      regbuf, valbuf);

(and another occurrence of similar return)   We're tryint to return a value
from a void function?   Is this perhaps another GCC-ism (sigh) that sneaked
in?

/bin/cc -c -g    -I. -I/play/gdb/src/gdb -I/play/gdb/src/gdb/config -DHAVE_CONFIG_H -I/play/gdb/src/gdb/../include/opcode -I/play/gdb/src/gdb/../readline/.. -I../bfd -I/play/gdb/src/gdb/../bfd  -I/play/gdb/src/gdb/../include -I../intl -I/play/gdb/src/gdb/../intl  -DUI_OUT=1   /play/gdb/src/gdb/i386-tdep.c
UX:acomp: ERROR: "/play/gdb/src/gdb/i386-tdep.c", line 739: void function cannot return value
UX:acomp: ERROR: "/play/gdb/src/gdb/i386-tdep.c", line 801: void function cannot return value