This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Make extra_order_size_table effective on targets with MAX_ALIGNMENT > 4


Richard Guenther wrote:
> On Wed, 21 Jun 2006, Mark Mitchell wrote:
> 
>> Richard Guenther wrote:
>>
>>> !       /* Build an alignment mask we can use for testing
>>> !          size % 2*align.  */
>> Why 2*align?
>>
>> I find this code very confusing.  Let's see if we can clear it up.  I
>> might be very dense, but maybe the next person will be too.
> 
> Ok, I added some more commentary to the code (see below for an updated
> patch).  

Thanks for taking the time to do this.

> !      Enforce alignment during lookup.  The resulting bin size must
> !      have the same or bigger alignment than the appearant alignment

s|appearant|apparent

(That might occur in a couple of places.)

OK with that change.

-- 
Mark Mitchell
CodeSourcery
mark@codesourcery.com
(650) 331-3385 x713


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]