GNU bug report logs - #40838
28.0.50; [feature/native-comp] Function overrides in init.el are not honored after deferred compilation

Previous Next

Package: emacs;

Reported by: Ihor Radchenko <yantar92 <at> gmail.com>

Date: Sat, 25 Apr 2020 08:19:02 UTC

Severity: normal

Found in version 28.0.50

Done: Andrea Corallo <akrl <at> sdf.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Ihor Radchenko <yantar92 <at> gmail.com>
To: Andrea Corallo <akrl <at> sdf.org>
Cc: 40838-done <at> debbugs.gnu.org
Subject: bug#40838: 28.0.50; [feature/native-comp] Function overrides in init.el are not honored after deferred compilation
Date: Tue, 28 Jul 2020 17:06:03 +0800
> That said this is an area I plan to fix to have it 100% compatible with
> the vanilla implementaiton once I'm done with moving all the eln file.
Great!

> My understanding was that the issue reported in this bug was not advise
> related but it could be also a side effect of?

Some of the broken functions were "redefined" with :around advice or
using el-patch. Though el-patch is working with function slots directly,
as I understand.

FYI, I do not encounter the original bug as often on the latest master.
What I noticed is that it also tends to happen when I change original
source code of the native-compiled functions and eval-sexp the new
versions. 

Best,
Ihor

Andrea Corallo <akrl <at> sdf.org> writes:

> Ihor Radchenko <yantar92 <at> gmail.com> writes:
>
>> Reopen.
>>
>> I think I found something related to this bug report in
>> exlplain-pause-mode package. See
>> https://github.com/lastquestion/explain-pause-mode/issues/65 for
>> details.
>>
>> best,
>> Ihor
>
> Hi Ihor,
>
> thanks for the finding.
>
> native-comp does not support advises to (most) primitive functions for
> performance reasons.  The elisp manual strongly discourage this practice
> tho.
>
> The issue can be worked around adding those functions into the
> `comp-never-optimize-functions' customize.
>
> That said this is an area I plan to fix to have it 100% compatible with
> the vanilla implementaiton once I'm done with moving all the eln file.
>
> My understanding was that the issue reported in this bug was not advise
> related but it could be also a side effect of?
>
> Regards
>
>   Andrea
>
> -- 
> akrl <at> sdf.org

-- 
Ihor Radchenko,
PhD,
Center for Advancing Materials Performance from the Nanoscale (CAMP-nano)
State Key Laboratory for Mechanical Behavior of Materials, Xi'an Jiaotong University, Xi'an, China
Email: yantar92 <at> gmail.com, ihor_radchenko <at> alumni.sutd.edu.sg




This bug report was last modified 4 years and 301 days ago.

Previous Next


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