Mark variables addressable if they are copied using libcall in RTL expander

Easwaran Raman eraman@google.com
Thu Jun 23 21:56:00 GMT 2011


On Thu, Jun 23, 2011 at 12:16 PM, Jakub Jelinek <jakub@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.

Thanks,
Easwaran
-------------- next part --------------
A non-text attachment was scrubbed...
Name: dse_fix.patch
Type: text/x-patch
Size: 1798 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20110623/0e064d2f/attachment.bin>


More information about the Gcc-patches mailing list