RE: [Patch,microblaze]: Optimized usage of reserved stack space for function arguments.

2015-07-06 Thread Ajit Kumar Agarwal
-Original Message- From: Oleg Endo [mailto:oleg.e...@t-online.de] Sent: Monday, July 06, 2015 7:07 PM To: Ajit Kumar Agarwal Cc: GCC Patches; Vinod Kathail; Shail Aditya Gupta; Vidhumouli Hunsigida; Nagaraju Mekala Subject: Re: [Patch,microblaze]: Optimized usage of reserved stack

Re: [Patch,microblaze]: Optimized usage of reserved stack space for function arguments.

2015-07-06 Thread Oleg Endo
Hi, Just some general comments... On 06 Jul 2015, at 22:05, Ajit Kumar Agarwal wrote: > +static bool > +microblaze_parm_needs_stack (cumulative_args_t args_so_far, tree type) > +{ > + enum machine_mode mode; 'enum' is not required in C++, please omit it. We've been trying

[Patch,microblaze]: Optimized usage of reserved stack space for function arguments.

2015-07-06 Thread Ajit Kumar Agarwal
. Along with the above conditions the stack space is not reserved if no arguments are passed. No regressions is seen in Deja GNU tests for microblaze. [Patch,microblaze]: Optimized usage of reserved stack space for function arguments. The changes are made in the patch for optimized