[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#59738: c-ts-mode is slow with large buffers.
From: |
Alan Mackenzie |
Subject: |
bug#59738: c-ts-mode is slow with large buffers. |
Date: |
Sat, 10 Dec 2022 21:34:20 +0000 |
Hello, Eli.
On Thu, Dec 08, 2022 at 22:37:05 +0200, Eli Zaretskii wrote:
> > From: Yuan Fu <casouri@gmail.com>
> > Date: Wed, 7 Dec 2022 16:40:15 -0800
> > Cc: Alan Mackenzie <acm@muc.de>,
> > 59738@debbugs.gnu.org
> > > Right, but with a long series of #define lines there should be no
> > > parse tree at all…
> > Ok, I think I know why. At the beginning of the file there is this
> > line
> > #ifndef _dce_12_0_SH_MASK_HEADER
> > So it’s parsed into a preproc_ifdef node, which contains every
> > #define directive in the file as its immediate child. Now you have
> > this node with a tons of immediate children. And querying this node
> > in font-lock is very slow, even with a limited range. I think for the
> > query result to be accurate, tree-sitter has to query the whole node
> > without considering the range, then throw away matches that are not
> > in the range.
> > Anyway, I activated my backup backup plan, which goes down the parse
> > tree to find a sufficiently small node to query. Now scrolling the
> > header file is fast as other files.
> Thanks, now c-ts-mode is twice as fast as c-mode with that file.
> Great job!
The bug which was causing it to be very slow is fixed, so I agree,
excellent job!
But I've measured it as being 62% faster (not twice as fast) as CC Mode.
A "normal" C file (xdisp.c) is around 160% faster, i.e. a little over 2½
times as fast. These timings are indeed significantly faster.
But given how slow CC Mode was held to be, is a factor 2.6 speed-up
really all that we were expecting from c-ts-mode? This is the sort of
speed-up one would get by replacing a 5 year old machine with a new one,
or using an optimised build in place of a debug build.
Was I perhaps a little unrealistic in expecting an order of magnitude
speed-up? Is there still scope for optimisation in c-ts-mode?
--
Alan Mackenzie (Nuremberg, Germany).
- bug#59738: c-ts-mode is slow with large buffers., Alan Mackenzie, 2022/12/01
- bug#59738: c-ts-mode is slow with large buffers., Yuan Fu, 2022/12/03
- bug#59738: c-ts-mode is slow with large buffers., Yuan Fu, 2022/12/06
- bug#59738: c-ts-mode is slow with large buffers., Eli Zaretskii, 2022/12/07
- bug#59738: c-ts-mode is slow with large buffers., Yuan Fu, 2022/12/07
- bug#59738: c-ts-mode is slow with large buffers., Eli Zaretskii, 2022/12/08
- bug#59738: c-ts-mode is slow with large buffers.,
Alan Mackenzie <=
- bug#59738: c-ts-mode is slow with large buffers., Yuan Fu, 2022/12/10
- bug#59738: c-ts-mode is slow with large buffers., Eli Zaretskii, 2022/12/11
- bug#59738: c-ts-mode is slow with large buffers., Alan Mackenzie, 2022/12/11
- bug#59738: c-ts-mode is slow with large buffers., Dmitry Gutov, 2022/12/11
- bug#59738: c-ts-mode is slow with large buffers., Eli Zaretskii, 2022/12/11
- bug#59738: c-ts-mode is slow with large buffers., Alan Mackenzie, 2022/12/11
- bug#59738: c-ts-mode is slow with large buffers., Eli Zaretskii, 2022/12/11
- bug#59738: c-ts-mode is slow with large buffers., Alan Mackenzie, 2022/12/11
- bug#59738: c-ts-mode is slow with large buffers., Eli Zaretskii, 2022/12/11
- bug#59738: c-ts-mode is slow with large buffers., Stefan Kangas, 2022/12/12