From: Charlie Somerville Date: 2013-05-20T18:05:00+09:00 Subject: [ruby-core:55082] Re: [ruby-trunk - Feature #8426] Implement class hierarchy method caching --5199e735_2eb141f2_168 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Content-Disposition: inline On Monday, 20 May 2013 at 5:28 PM, funny_falcon (Yura Sokolov) wrote: > Charlie, why sa_index_t is uint64_t ? it really should be 32bit for better CPU cache locality. > Yes, it will limits ID to 32bit values, but ID should not increase to greater values, > otherwise it is a memory leak. Sorry, this was an oversight. I've pushed a commit to make sa_index_t 32 bit. --5199e735_2eb141f2_168 Content-Type: text/html; charset="utf-8" Content-Transfer-Encoding: quoted-printable Content-Disposition: inline
On M= onday, 20 May 2013 at 5:28 PM, funny=5Ffalcon (Yura Sokolov) wrote:
Charlie, why sa=5Findex=5Ft is uint64=5Ft =3F it real= ly should be 32bit for better CPU cache locality.
Yes= , it will limits ID to 32bit values, but ID should not increase to greate= r values,
otherwise it is a memory leak.
Sorry, this was an oversight. I've pus= hed a commit to make sa=5Findex=5Ft 32 bit.
--5199e735_2eb141f2_168--