This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Fix complex argument handling in ppc64 dummy function call


(not a real review, probably Ulrich would be
the best reviewer).

On 03/01/2013 05:08 PM, Tiago Stürmer Daitx wrote:
> +		  struct type *target_type;
> +
> +		  target_type = check_typedef (TYPE_TARGET_TYPE (type));
> +                  if (write_pass)
> +		    ppc64_sysv_abi_push_float (gdbarch, regcache, tdep,

While quickly skimming the patch I just noticed something wrong with
the indenting here.

> +					       target_type, val + i *
> +					       TYPE_LENGTH (target_type),
> +					       freg, greg, gparam);
> +                  freg += 2;
> +                  greg += 2;
> +                  gparam = align_up (gparam + TYPE_LENGTH (target_type),
> +                                     tdep->wordsize);
> +                }


-- 
Pedro Alves


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]