Opened 7 years ago

Closed 7 years ago

#1277 closed defect (fixed)

[awaiting confirmation] two further flint spkg problems

Reported by: was Owned by: was
Priority: major Milestone: sage-2.10
Component: packages: standard Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

On Nov 25, 2007 9:52 PM, Bill Hart <goodwillhart@googlemail.com> wrote:
> 
> This is a compiler bug which we've seen before. On Itanium the build
> should not be using the -funroll-loops flag.
> 
> I guess we need to check for ia64 when setting the compiler flags?

The culprit is this line, which is now in patches/makefile:

CFLAGS = $(INCS) -funroll-loops -fexpensive-optimizations $(FLINT_TUNE) -O3^M

On ia64 FLINT_TUNE is properly set to not include funroll-loops, but for some dumb
reason -funroll-loops is put in the CFLAGS explicitly in the patches/makefile included
in Sage.  

Changing the above line to 

CFLAGS = $(INCS)  -fexpensive-optimizations $(FLINT_TUNE) -O3^M

completely fixes the problem. 

As an aside, I don't think the .svn subdirectories should be included with
with flint-*.spkg.  

A new flint spkg is at 

http://sage.math.washington.edu/home/was/tmp/flint-0.9-r1075.p2.spkg

Change History (3)

comment:1 Changed 7 years ago by mabshoff

  • Summary changed from two further flint spkg problems to [awaiting conformation] two further flint spkg problems

The new FLINT.spkg at

http://sage.math.washington.edu/home/mabshoff/flint-1.0.spkg

should solve the problem.

Cheers,

Michael

comment:2 Changed 7 years ago by rlm

  • Summary changed from [awaiting conformation] two further flint spkg problems to [awaiting confirmation] two further flint spkg problems

comment:3 Changed 7 years ago by mabshoff

  • Resolution set to fixed
  • Status changed from new to closed

This was fixed in the Sage 2.9.x time frame, as confirmed by Kate in https://groups.google.com/group/sage-devel/t/40538e2d4e6742dd

Cheers,

Michael

Note: See TracTickets for help on using tickets.