Re: i386 and x86_64 fenv support

Corinna Vinschen <[email protected]>
Newsgroups gmane.comp.lib.newlib
Message-ID <[email protected]>
On Aug 27 08:31, Joel Sherrill wrote:
> Hi
> 
> Earlier we discussed moving the winsup fenv.h files to newlib. Splitting
> winsup/cygwin/include/fenv.h to create a sys/fenv.h looks easy enough.
> But I have some status and a question after tinkering with this code out
> of tree and compiling with the RTEMS i386 and x86_64
> 
> It includes winsup.h and wincap.h but it doesn't need them.

Yeah, that's kind of a boilerplate include in Cygwin code.  Just scratch
it, it can't be overly important for the fenv functionality.

> [...]
> There are prototypes for fegetprec() and fesetprec(). I think the
> prototypes need to move to the shared fenv.h and commented
> stubs for the constants needs to be in the shared dummy sys/fenv.h.
> Does this sound right?

As far as I can see, none of Linux or BSD have these functions.  It may
be the right thing to do to keep them local to Cygwin since they never
made it into a standard or one of the systems we're interested in here.

Ideally they are never defined in a header, not even on Cygwin.  We only
have to keep exporting these symbols in Cygwin for backward compat.

> My remaining issue is that it appears that i386 and x86_64 will need to
> share the same files. Do I just copy the files to both machine directories?
> I don't know if newlib's build infrastructure supports putting it in one
> place and using it in another architecture.

Shouldn't a symlink from x86_64/file to ../i386/file work?  git handles
symlinks just fine.



Corinna

-- 
Corinna Vinschen
Cygwin Maintainer
Red Hat
signature.asc (application/pgp-signature, 833 B)
-----BEGIN PGP SIGNATURE-----

iQIzBAEBCAAdFiEEoVYPmneWZnwT6kwF9TYGna5ET6AFAl1lTqoACgkQ9TYGna5E
T6C0rhAAiuCad/YuRulqNbb/nY0yyopyLJBt1p8bewr4LHxIM+NvPf/iXi/9K+tJ
Ok/NcLiNcX7Cjnq7yvetOfuTNb23149+7/yXHw8YRXewwu5AS/dz2nOUe8rvclwQ
733vR75z8ts6ijZvXmooGnsWVzqMjdQ550tgd+c8HDOzCHqvbdt16qBxJqA3R+mI
5GyL2AACwi2T6IhhT8Kq2hI/ml4Zw0RYKfI3cn6G+L+UXumB93l73BFoGYChnc1g
qwidkstLT+qFIHqmNCE7e4yzoluiKMeJ+iGusbe/t1bUubrN8g5y2x/jMf1IqNrh
a3BxhR7o4QywOY8AMR9P40CEaab6P+NDxuL7X6BcPikzbZhmuRCGu1Gs0aLkHBwE
jNkGNI4ZFBOVU2+lJrfh4YUmSsfHRmTFTFn8BzPvvs8OpZPU0rSGXbBpV5gWp8nQ
CH/ZC9Lp1GsbPUDqwWWsvueFex7NZmiwqmpZrubOMcjYbU2pN2ZtQqedsYGmQi0H
mCOo+uaeSV3M6oTk/JBWf0KNhV15OfPtYow3tc1HCTh4HSUHuiGfR68V8Lt11jsH
uFYCHz3TJGyixi3xWZ6bXss3Jd6G5BWz88MRwZhW9Aio+EFgXAmxCW2rY5LfL/FP
phv5v7fVoU0OXJmhYEQMNDT7JgqEfl8pQHWoJEyVe608sTldWo4=
=olDk
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.