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
>
