Richard Henderson <r...@redhat.com> writes:
> On 08/03/2014 03:39 AM, Richard Sandiford wrote:
>> +struct rtx_subrtx_bound_info {
>> + unsigned char start;
>> + unsigned char count;
>> +};
>
> Given this structure is only two bytes...
>
>> + /* The bounds to use for iterating over subrtxes. */
>> + const rtx_subrtx_bound_info *m_bounds;
>
> ... wouldn't it be better to pass by value instead of by reference?
This shows I should expand the comment, but m_bounds is an array indexed
by rtx code.
Thanks,
Richard
