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

make sure Julia doesn't use x87 math. #43978

Merged
merged 13 commits into from
Feb 8, 2022
Merged

make sure Julia doesn't use x87 math. #43978

Changes from all commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
f2e84c7
properly link libjulia to openlibm
simeonschaub Jan 29, 2022
a2a2d73
add `$(LIBM)` before llvm
simeonschaub Jan 31, 2022
cadcfdb
attempt a fix
oscardssmith Feb 2, 2022
f4ff233
next try
oscardssmith Feb 3, 2022
3b6deef
try 3
oscardssmith Feb 3, 2022
7de3e91
take 4
oscardssmith Feb 3, 2022
453eb36
take 5
oscardssmith Feb 3, 2022
c305f90
fix gcc build
oscardssmith Feb 7, 2022
cef4699
fix whitespace
oscardssmith Feb 7, 2022
db718ec
fix that won't mess with clang (hopefully)
oscardssmith Feb 8, 2022
d2c538b
only use fpmath=sse on i386
oscardssmith Feb 8, 2022
c9dd37c
use on all x86
oscardssmith Feb 8, 2022
62631ba
only use on 32 bit
oscardssmith Feb 8, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions Make.inc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -874,6 +874,10 @@ ifneq (,$(filter $(ARCH), powerpc64le ppc64le))
DIST_ARCH:=ppc64le
endif
ifeq (1,$(ISX86))
# on x86 make sure not to use 80 bit math when we want 64 bit math.
ifeq (32,$BINARY))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

missing a ( here

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#44081

JCFLAGS += -mfpmath=sse
endif
DIST_ARCH:=$(BINARY)
endif
ifneq (,$(findstring arm,$(ARCH)))
Expand Down