avr-libc-dev
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [avr-libc-dev] libm/fplib/irexp.S refers to "fplib-as.inc"


From: Joerg Wunsch
Subject: Re: [avr-libc-dev] libm/fplib/irexp.S refers to "fplib-as.inc"
Date: Thu, 9 Jan 2003 12:37:10 +0100
User-agent: Mutt/1.2.5i

As Philip Søeberg wrote:

> Just noticed that the file libm/fplib/irexp.S includes one
> "fplib-as.inc" which I cannot find in CVS?

Most likely, fplib.inc is meant.

Obviously :), this file is not built as part of avr-libc.  The
function is neither documented nor standard.  Perhaps it aims
to be (int)frexp(), but i don't see much point in having it.

Even with fplib.inc, i doubt it will work at all.  A branch is
targeted to go to _frexp_200, but the code after _irexp_200 in this
file is unused.  It /looks/ like the branch should go there instead.
(It will assemble and link anyway, since most of fplib still bogusly
uses globally known labels, and incidentally, frexp.S contains a label
named _frexp_200.  Not declaring external labels before use is
non-fatal in Unix assemblers, so the jump to outside the own file will
remain unnoticed.)

I'm currently not in the mood of thinking whether fixing this will
make it work.  I think we should simply drop that file instead.

-- 
J"org Wunsch                                           Unix support engineer
address@hidden        http://www.interface-systems.de/~j/




reply via email to

[Prev in Thread] Current Thread [Next in Thread]