I've created a patch for sympow, which the release manager has agreed to merge in 4.5.3 - in fact, he has kindly made it a blocker!!

He is happy with it, but wants someone able to comment on the changes I made to the file fpu.c.

This is the original file.

http://boxen.math.washington.edu/home/kirkby/patches/sympow-1.018.1.p8/src/fpu.c

which sets some bits in the control register for the floating point processor. However, it uses Linux constructs and includes a linux header file.

Here's the changed version, which I used inline assembly code.

http://boxen.math.washington.edu/home/kirkby/patches/sympow-1.018.1.p8/patches/fpu.c

The file is pretty well commented, but Mitesh feels unable to review that particular file.

You might ask why did I read the register, then change the bits, then write it, rather than just write a number as the original sympow code does.

The reason is that one of those bits in the register is reserved by Intel, so I don't think one should write to a reserved bit. It's better to read it, change the two bits I needed to, then write it back.

Please add comments on #9703

Dave

--
To post to this group, send an email to [email protected]
To unsubscribe from this group, send an email to 
[email protected]
For more options, visit this group at http://groups.google.com/group/sage-devel
URL: http://www.sagemath.org

Reply via email to