#21064: Enable NTL's '-march=native' more cautiously
-------------------------------------+-------------------------------------
Reporter: leif | Owner: leif
Type: defect | Status: needs_review
Priority: blocker | Milestone: sage-7.3
Component: packages: | Resolution:
standard |
Keywords: assembler shlx | Merged in:
mulx |
Authors: Leif Leonhardy | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/leif/enable_NTL_native_more_safely|
3cccfbbe23776c2fc682a6dce73b1efee23a8339
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by leif):
Replying to [comment:13 jdemeyer]:
> That's a lot of complicated logic which looks quite fragile.
It's about 30 lines of code, 10 of them `echo`s. And I fail to see where
it's "fragile", as we don't do dangerous things here, just try to not lose
performance without reason.
\\
> Why not simply:
>
> 1. Try to build normally.
>
> 2. If that failed, retry without `-march=native`.
That's nearly dumber than simply disabling `NATIVE`; if NTL took a few
seconds (maybe minutes) to build, I'd perhaps agree. And we cannot be
sure it will fail ''early''.
Imagine we'd do so for every package, probably trying more than one
option.
As I said, in the long run, proper tests belong to NTL's `configure`, and
the general problem on MacOS X will be solved on #20779. I'd also
appreciate if we had an (optional) binutils package for older Linux
distros, otherwise we should perhaps stop building our own GCC (on those
at least).
--
Ticket URL: <https://trac.sagemath.org/ticket/21064#comment:14>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
