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

Jump to bottom

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 1 commit
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
2 changes: 1 addition & 1 deletion Make.inc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -892,7 +892,7 @@ OPENBLAS_TARGET_ARCH:=ARMV7
endif

# on x86 make sure not to use 80 bit math when we want 64 bit math.
ifeq ($(BUILD_ARCH),i386)
ifeq (1,$(ISX86))
oscardssmith marked this conversation as resolved.
Show resolved Hide resolved
JCFLAGS += -mfpmath=sse
endif
# If we are running on aarch64 (e.g. ARMv8 or ARM64), set certain options automatically
Expand Down