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
>

Reply via email to