Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove <object> cast in real_mpfi.pyx #12829

Closed
jdemeyer opened this issue Apr 11, 2012 · 4 comments
Closed

Remove <object> cast in real_mpfi.pyx #12829

jdemeyer opened this issue Apr 11, 2012 · 4 comments
Assignees
@sagetrac-drkirkby
Labels
c: porting p: critical / 2 t: bug
Milestone
sage-5.0

Comments

@jdemeyer
Copy link
Contributor

The following is very wrong code (from sage/rings/real_mpfi.pyx)

# todo: The <object> coerce is evidently to get around a weird bug in SageX (?)   
mpfi_interv_q(self.value, <object>rat.value, <object>rat1.value)

Surprisingly, it only breaks on PPC64. See #11705 for more context.

Component: porting

Keywords: ppc64 big endian mpfi

Author: Paul Zimmermann, Jeroen Demeyer

Reviewer: François Bissey

Merged: sage-5.0.beta14

Issue created by migration from https://trac.sagemath.org/ticket/12829
@jdemeyer jdemeyer added this to the sage-5.0 milestone Apr 11, 2012
@jdemeyer
Copy link
Contributor Author

Attachment: 12829_mpfi.patch.gz

@kiwifb
Copy link
Member

kiwifb commented Apr 11, 2012

comment:2

Works for me on the power7 box I have access to.

@jdemeyer
Copy link
Contributor Author

Reviewer: François Bissey

@jdemeyer
Copy link
Contributor Author

Merged: sage-5.0.beta14

@nexttime nexttime mannequin mentioned this issue Apr 25, 2012
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sagetrac-drkirkby sagetrac-drkirkby

Labels
c: porting p: critical / 2 t: bug
Projects
None yet
Milestone
sage-5.0
Development

No branches or pull requests
2 participants
@kiwifb @jdemeyer