From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Sun, 5 Dec 2010 15:58:49 +0300 From: Alexey Tourbin To: ALT Linux Team development discussions Message-ID: <20101205125849.GG25886@altlinux.org> References: <1290722666-24606-1-git-send-email-kirill@shutemov.name> <20101205012451.GD25886@altlinux.org> <20101205123948.GN24222@osdn.org.ua> Mime-Version: 1.0 Content-Type: text/plain; charset=koi8-r Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20101205123948.GN24222@osdn.org.ua> Subject: Re: [devel] [PATCH 0/3] optimize rpmsetcmp() X-BeenThere: devel@lists.altlinux.org X-Mailman-Version: 2.1.12 Precedence: list Reply-To: ALT Linux Team development discussions List-Id: ALT Linux Team development discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 05 Dec 2010 12:58:49 -0000 Archived-At: List-Archive: List-Post: On Sun, Dec 05, 2010 at 02:39:48PM +0200, Michael Shigorin wrote: > On Sun, Dec 05, 2010 at 04:24:51AM +0300, Alexey Tourbin wrote: > > The explanation is: > > Don't try to improve my code. > > Дык тормозит же ж. И на сборочнице, и на localhost. > Приходится дольше сидеть в коридоре у розетки, > чтоб собрать тестовую исошку. 4.0.4-alt100.3 -> 4.0.4-alt100.5 примерно в 4 раза быстрее. Меркантильные соображения тут не нужны. Вопрос по сути кода остается - стоит ли переходить от 'char bitv[]' к битовой шкале. С точки зрения прозрачности кода - не стоит. С точки зрения скорости исполнения - выгода заметная, но не офигительная. К тому же Кирилл там использует целочисленное деление, а оно выполняется не везде дёшево. Хотя, с константой в знаменателе, gcc скорее всего заменяет деление на сдвиг. Но это уже детали. > -- > ---- WBR, Michael Shigorin > ------ Linux.Kiev http://www.linux.kiev.ua/