Opened 3 years ago

Closed 6 months ago

#29302 closed defect (invalid)

ecm build failure on cygwin-minimal

Reported by: mkoeppe Owned by:
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: porting Keywords:
Cc: embray Merged in:
Authors: Reviewers: Kwankyu Lee
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

https://github.com/mkoeppe/sage/runs/493864794

libtool: compile:  gcc -DHAVE_CONFIG_H -I. -I./x86_64 -I/cygdrive/d/a/sage/sage/local/include -I/cygdrive/d/a/sage/sage/local/include -g -march=native -g -O3 -fPIC -c ecm.c  -DDLL_EXPORT -DPIC -o .libs/libecm_la-ecm.o
In file included from sp.h:33,
                 from ecm-impl.h:29,
                 from ecm.c:26:
ecm-gmp.h:134:8: warning: '__gmpn_redc_n' redeclared without dllimport attribute: previous dllimport ignored [-Wattributes]
  134 |   void __gmpn_redc_n (mp_ptr, mp_ptr, mp_srcptr, mp_size_t, mp_srcptr);
      |        ^~~~~~~~~~~~~
/tmp/cco2lRak.s: Assembler messages:
/tmp/cco2lRak.s:1351: Error: invalid register for .seh_savexmm
/tmp/cco2lRak.s:1776: Error: invalid register for .seh_savexmm
make[6]: *** [Makefile:1085: libecm_la-ecm.lo] Error 1
make[6]: Leaving directory '/cygdrive/d/a/sage/sage/local/var/tmp/sage/build/ecm-7.0.4.p1/src'

Change History (10)

comment:1 Changed 3 years ago by mkoeppe

This failure seems to depend on the CPU where it gets to run. It built correctly on cygwin-minimal at https://github.com/mkoeppe/sage/runs/518106009

comment:2 Changed 3 years ago by mkoeppe

Milestone: sage-9.1sage-9.2
Status: newneeds_info

comment:3 Changed 2 years ago by mkoeppe

Milestone: sage-9.2sage-9.3

comment:4 Changed 22 months ago by mkoeppe

Milestone: sage-9.3sage-9.4

Moving this ticket to 9.4, as it seems unlikely that it will be merged in 9.3, which is in the release candidate stage

comment:5 Changed 18 months ago by mkoeppe

Milestone: sage-9.4sage-9.5

comment:6 Changed 14 months ago by mkoeppe

Milestone: sage-9.5sage-9.6

Stalled in needs_review or needs_info; likely won't make it into Sage 9.5.

comment:7 Changed 10 months ago by mkoeppe

Milestone: sage-9.6sage-9.7

comment:8 Changed 6 months ago by mkoeppe

Milestone: sage-9.7sage-duplicate/invalid/wontfix
Status: needs_infoneeds_review

outdated

comment:9 Changed 6 months ago by klee

Reviewers: Kwankyu Lee
Status: needs_reviewpositive_review

I agree.

comment:10 Changed 6 months ago by mkoeppe

Resolution: invalid
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.