Ticket #1277 (closed defect: fixed)

Opened 6 years ago

Last modified 5 years ago

[awaiting confirmation] two further flint spkg problems

Reported by: was Owned by: was
Priority: major Milestone: sage-2.10
Component: packages: standard Keywords:
Cc: Work issues:
Report Upstream: Reviewers:
Authors: Merged in:
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

comment:1 Changed 5 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 5 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 5 years ago by mabshoff

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

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.