Quantcast

Re: [Emacs-diffs] master b389379: bytecomp: Don't inline functions that use byte-switch (Bug#26518)

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: [Emacs-diffs] master b389379: bytecomp: Don't inline functions that use byte-switch (Bug#26518)

Stefan Monnier
> +    ;; optimized switch bytecode makes it impossible to guess the correct
> +    ;; `byte-compile-depth', which can result in incorrect inlined code.

I really doubt this is true.  Maybe it makes it difficult/impractical,
but I doubt it makes it impossible.


        Stefan

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: [Emacs-diffs] master b389379: bytecomp: Don't inline functions that use byte-switch (Bug#26518)

Vibhav Pant-2
On Thu, Apr 20, 2017 at 10:47 PM, Stefan Monnier
<[hidden email]> wrote:
> I really doubt this is true.  Maybe it makes it difficult/impractical,
> but I doubt it makes it impossible.

I'll try adding a longish outline of the problem to the comment, and
a TODO on investigating a fix.

--
Vibhav Pant
[hidden email]

Loading...