Re: [scratch/igc] 985247b6bee crash on Linux, KDE, Wayland

[Eval EXEC]

Eli Zaretskii <eliz@gnu.org> writes:

>> From: Eval EXEC <execvy@gmail.com>
>> Cc: emacs-devel <emacs-devel@gnu.org>
>> Date: Thu, 05 Sep 2024 18:44:40 +0800
>> 
>> Helmut Eller <eller.helmut@gmail.com> writes:
>> 
>> > On Thu, Sep 05 2024, Eval EXEC wrote:
>> >
>> >> Helmut Eller <eller.helmut@gmail.com> writes:
>> >>
>> >>> I think you need to add -fno-omit-frame-pointer (or use the patch
>> >>> https://github.com/Ravenbrook/mps/pull/38).  Without that, MPS doesn't
>> >>> trace the %rbp register properly.
>> >>
>> >> Will `-fno-omit-frame-pointer`
>> >
>> > This will cost a little.  Though, it usually also helps perf to produce
>> > more useful reports.
>> >
>> >> or apply use github PR patch performance
>> >> emacs's performance?
>> >
>> > The patch should not cost any performance.
>> 
>> I compiled scratch/igc emacs with  `-fno-omit-frame-pointer`, it crash
>> again, and I don't know how to reproduce.
>
> Please show the value of system-configuration-options, so that we are
> sure the switch was indeed noted.

system-configuration-options is a variable defined in emacs.c.

Value
"'CFLAGS=-O3 -fno-omit-frame-pointer -mtune=native -march=native' 
--prefix=/home/exec/Projects/git.savannah.gnu.org/git/emacs-build/scratch_igc 
--with-mps=yes --with-imagemagick --with-modules --without-compress-install 
--with-native-compilation --with-mailutils --enable-link-time-optimization 
--with-tree-sitter --with-xinput2 --with-dbus --with-native-compilation=aot 
--with-file-notification=inotify"


> Also, I would suggest to rebuild with -O2, since -O3e could perform
> other unsafe optimizations.

OK, I'm going to replace -O3 by -O2 now.
--