On Thu, Jun 23, 2011 at 11:47 PM, Easwaran Raman <era...@google.com> wrote: > On Thu, Jun 23, 2011 at 12:16 PM, Jakub Jelinek <ja...@redhat.com> wrote: >> On Thu, Jun 23, 2011 at 12:02:35PM -0700, Easwaran Raman wrote: >>> + if (y_expr) >>> + mark_addressable (y_expr); >> >> Please watch formatting, a tab should be used instead of 8 spaces. >> >>> + if (x_expr) >>> + mark_addressable (x_expr); >> >> Ditto. >> >>> @@ -1084,6 +1084,8 @@ initialize_argument_information (int num_actuals A >>> && TREE_CODE (base) != SSA_NAME >>> && (!DECL_P (base) || MEM_P (DECL_RTL (base))))) >>> { >>> + mark_addressable (args[i].tree_value); >>> + >> >> Likewise, plus the line is indented too much, each level should be indented >> by 2 chars. >> >> Jakub >> > > I have attached a new patch that fixes the formatting issues.
Ok. Thanks, Richard. > Thanks, > Easwaran >