generic-library/bionic
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Use gcc to build and test x86_64 long double code.

Browse Source 
Clang/llvm x86_64 configuration of long double is still
incompatible with gcc.
https://llvm.org/bugs/show_bug.cgi?id=23897

BUG: 21899249
Change-Id: I47fd6d0ce2daa84ae0035e7885a4d90e4c3056f7
This commit is contained in:
Chih-Hung Hsieh 2015-06-23 14:18:14 -07:00 committed by Chih-hung Hsieh
parent a8993c994e
commit e79d06b0e8
3 changed files with 20 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
libc/Android.mk
View File

@ -580,6 +580,13 @@ libc_common_cflags += -DTARGET_USES_LOGD
endif endif
use_clang := $(USE_CLANG_PLATFORM_BUILD) use_clang := $(USE_CLANG_PLATFORM_BUILD)
# Clang/llvm has incompatible long double (fp128) for x86_64.
# https://llvm.org/bugs/show_bug.cgi?id=23897
ifeq ($(TARGET_ARCH),x86_64)
use_clang := false
endif
ifeq ($(use_clang),) ifeq ($(use_clang),)
use_clang := false use_clang := false
endif endif

10
libm/Android.mk
View File

@ -3,6 +3,12 @@ LOCAL_PATH:= $(call my-dir)
bionic_coverage := false bionic_coverage := false
# Clang/llvm has incompatible long double (fp128) for x86_64.
# https://llvm.org/bugs/show_bug.cgi?id=23897
ifeq ($(TARGET_ARCH),x86_64)
libm_clang := false
endif
# ----------------------------------------------------------------------------- # -----------------------------------------------------------------------------
# libm.a # libm.a
# ----------------------------------------------------------------------------- # -----------------------------------------------------------------------------
@ -473,7 +479,7 @@ LOCAL_C_INCLUDES_x86 += $(LOCAL_PATH)/i387
LOCAL_C_INCLUDES += $(LOCAL_PATH)/upstream-freebsd/lib/msun/src/ LOCAL_C_INCLUDES += $(LOCAL_PATH)/upstream-freebsd/lib/msun/src/
LOCAL_C_INCLUDES_64 += $(LOCAL_PATH)/upstream-freebsd/lib/msun/ld128/ LOCAL_C_INCLUDES_64 += $(LOCAL_PATH)/upstream-freebsd/lib/msun/ld128/
LOCAL_CLANG := true LOCAL_CLANG := $(libm_clang)
LOCAL_ARM_MODE := arm LOCAL_ARM_MODE := arm
LOCAL_CFLAGS := \ LOCAL_CFLAGS := \
-DFLT_EVAL_METHOD=0 \ -DFLT_EVAL_METHOD=0 \
@ -514,7 +520,7 @@ LOCAL_LDFLAGS_x86 := -Wl,--hash-style=both
LOCAL_LDFLAGS := -Wl,--version-script,$(LOCAL_PATH)/libm.map LOCAL_LDFLAGS := -Wl,--version-script,$(LOCAL_PATH)/libm.map
LOCAL_MODULE := libm LOCAL_MODULE := libm
LOCAL_CLANG := true LOCAL_CLANG := $(libm_clang)
LOCAL_SYSTEM_SHARED_LIBRARIES := libc LOCAL_SYSTEM_SHARED_LIBRARIES := libc
LOCAL_WHOLE_STATIC_LIBRARIES := libm LOCAL_WHOLE_STATIC_LIBRARIES := libm

5
tests/Android.mk
View File

@ -137,6 +137,11 @@ libBionicStandardTests_static_libraries := \
libBionicStandardTests_ldlibs_host := \ libBionicStandardTests_ldlibs_host := \
-lrt \ -lrt \
# Clang/llvm has incompatible long double (fp128) for x86_64.
# https://llvm.org/bugs/show_bug.cgi?id=23897
# This affects most of math_test.cpp.
libBionicStandardTests_clang_target := false
module := libBionicStandardTests module := libBionicStandardTests
module_tag := optional module_tag := optional
build_type := target build_type := target