GNU bug report logs - #6365
bidi data structure inefficiencies

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Dan Nicolaescu <dann <at> gnu.org>
Subject: bug#6365: closed (Re: bug#6365: bidi data structure inefficiencies)
Date: Thu, 06 Oct 2011 21:23:02 +0000

[Message part 1 (text/plain, inline)]

Your bug report

#6365: bidi data structure inefficiencies

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 6365 <at> debbugs.gnu.org.

-- 
6365: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=6365
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Glenn Morris <rgm <at> gnu.org>
To: 6365-done <at> debbugs.gnu.org
Subject: Re: bug#6365: bidi data structure inefficiencies
Date: Thu, 06 Oct 2011 17:22:40 -0400

Eli Zaretskii wrote:

>>   https://lists.gnu.org/archive/html/emacs-devel/2010-06/msg00164.html
>> 
>> AFAIU, the conclusion was that there are no evident optimizations, and
>> that only profiling the current code against an alternative can tell
>> which one is better.
>> 
>> FWIW, I don't plan working o this any time soon.
>
> Oh, and I did convert 2 members of `struct bidi_it' from `int's to
> 1-bit fields.

OK; doesn't seem worth keeping this open as a bug then.

[Message part 3 (message/rfc822, inline)]

From: Dan Nicolaescu <dann <at> gnu.org>
To: bug-gnu-emacs <at> gnu.org
Subject: bidi data structure inefficiencies
Date: Sun, 06 Jun 2010 23:21:32 -0400

Some bidi data structures are bigger that they need to be, this
probably results in additional cache misses.
Examples:

struct bidi_saved_info could  use bitfields for the bidi_type_t members
Same for bidi_stack

bidi_it could use bitfields for a lot of it's members.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.