[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#14892: [PATCH] Enhance Elisp compare functions (< <= > >=) to take v
From: |
Stefan Monnier |
Subject: |
bug#14892: [PATCH] Enhance Elisp compare functions (< <= > >=) to take var args |
Date: |
Tue, 10 Sep 2013 16:16:02 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.3.50 (gnu/linux) |
>> Looking at your patch a second time, I don't see how/where it
>> handles (< a b c) in byte-compiled code: the byte-codes only
>> handle 2-arguments, and the byte-compiler is left unchanged.
> (disassemble (byte-compile (lambda () (< x y))))
> byte code:
> args: nil
> 0 varref x
> 1 varref y
> 2 lss
> 3 return
> (disassemble (byte-compile (lambda () (< x y z))))
> byte code:
> args: nil
> 0 constant <
> 1 varref x
> 2 varref y
> 3 varref z
> 4 call 3
> 5 return
Ha! Indeed, it does! great!
So could you change the bytecode.c code along the lines I suggested so
as to eliminate the slight speed penalty?
Stefan
- bug#14892: [PATCH] Enhance Elisp compare functions (< <= > >=) to take var args, Barry OReilly, 2013/09/06
- bug#14892: [PATCH] Enhance Elisp compare functions (< <= > >=) to take var args, Stefan Monnier, 2013/09/06
- bug#14892: [PATCH] Enhance Elisp compare functions (< <= > >=) to take var args, Barry OReilly, 2013/09/09
- bug#14892: [PATCH] Enhance Elisp compare functions (< <= > >=) to take var args, Stefan Monnier, 2013/09/10
- bug#14892: [PATCH] Enhance Elisp compare functions (< <= > >=) to take var args, Barry OReilly, 2013/09/10
- bug#14892: [PATCH] Enhance Elisp compare functions (< <= > >=) to take var args,
Stefan Monnier <=
- bug#14892: [PATCH] Enhance Elisp compare functions (< <= > >=) to take var args, Barry OReilly, 2013/09/10
- bug#14892: [PATCH] Enhance Elisp compare functions (< <= > >=) to take var args, Stefan Monnier, 2013/09/10
- bug#14892: [PATCH] Enhance Elisp compare functions (< <= > >=) to take var args, Barry OReilly, 2013/09/11
- bug#14892: [PATCH] Enhance Elisp compare functions (< <= > >=) to take var args, Stefan Monnier, 2013/09/11