Re: macOS metal rendering engine in mac port

[Eli Zaretskii]

> From: Aaron Jensen <aaronjensen@gmail.com>
> Date: Mon, 24 May 2021 10:58:23 -0700
> Cc: emacs-devel@gnu.org, Alan Third <alan@idiocy.org>, 
>       YAMAMOTO Mitsuharu <mituharu@math.s.chiba-u.ac.jp>
> 
> On Mon, May 24, 2021 at 9:43 AM Eli Zaretskii <eliz@gnu.org> wrote:
> >
> > Of course.  But it also does that for every other face, starting with
> > font-lock faces.  So something is still amiss here, because 40 is too
> > small a number.
> 
> I'm not seeing this. With line numbers off the only call to
> merge_faces is for escape-glyph when I use scroll-up-benchmark. I'm on
> Alan's surface-stuff branch if that makes any difference.
> 
> > > > Also, what did Emacs do when you collected these numbers, and in which
> > > > buffer under what major mode?
> > >
> > > emacs -Q src/xdisp.c with scroll-up-benchmark
> >
> > Then you should see font-lock-comment-face, font-lock-keyword-face,
> > font-lock-variable-name-face, etc., and calls to merge_face_ref for
> > them coming from face_at_buffer_position.
> 
> Yeah, I don't see calls to merge_faces for these. Here's the debug code I 
> added:

I said merge_face_ref, not merge_faces.  See face_at_buffer_position,
which is called whenever the display code finds a new face at some
buffer position.

I don't know what that branch does; I thought we were talking about
the master branch.