Fixed. I removed the bogus inline forms -- which leaves the moderately inefficient ones. It would be nice to improve the inline code to handle also floor with fixnums and multiple return values.<br><br><div class="gmail_quote">
On Mon, Feb 8, 2010 at 2:00 PM, Alexander Gavrilov <span dir="ltr"><<a href="mailto:angavrilov@gmail.com">angavrilov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Hi, there is a bug in code inlining:<br>
<br>
(defun test (a b)<br>
  (declare (fixnum a b))<br>
  (multiple-value-bind (q r) (floor a b)<br>
    (declare (fixnum q r))<br>
    (print q)<br>
    (print r)))<br>
<br>
> (test 3 2)<br>
<br>
1<br>
1<br>
1<br>
<br>
> (compile 'test)<br>
> (test 3 2)<br>
<br>
NIL cannot be coerced to a C int.<br>
<br>
<br>
This happens because only the calculation for the 0th value is<br>
inlined, and thus r contains garbage.<br>
<br>
<br>
On Wed, Feb 3, 2010 at 11:43 PM, Juan Jose Garcia-Ripoll<br>
<div class="im"><<a href="mailto:juanjose.garciaripoll@googlemail.com">juanjose.garciaripoll@googlemail.com</a>> wrote:<br>
</div><div><div></div><div class="h5">> I want to produce a new release. For that I am officially beginning a test<br>
> phase -- I will have to update the test farm, unable yet to install Samium's<br>
> new system --, and will look into the following problems<br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Instituto de Física Fundamental, CSIC<br>c/ Serrano, 113b, Madrid 28006 (Spain) <br><a href="http://juanjose.garciaripoll.googlepages.com">http://juanjose.garciaripoll.googlepages.com</a><br>