Skip to content

Commit

Permalink
[builtins] Implement rounding mode support for i386/x86_64
Browse files Browse the repository at this point in the history
Differential Revision: https://reviews.llvm.org/D69870
  • Loading branch information
kongy committed Nov 18, 2019
1 parent 4624e83 commit 1689ad2
Show file tree
Hide file tree
Showing 4 changed files with 44 additions and 2 deletions.
1 change: 1 addition & 0 deletions compiler-rt/lib/builtins/CMakeLists.txt
Expand Up @@ -239,6 +239,7 @@ set(x86_ARCH_SOURCES
floatuntixf.c
mulxc3.c
powixf2.c
i386/fp_mode.c
)

if (NOT MSVC)
Expand Down
39 changes: 39 additions & 0 deletions compiler-rt/lib/builtins/i386/fp_mode.c
@@ -0,0 +1,39 @@
//===----- lib/i386/fp_mode.c - Floaing-point mode utilities -----*- C -*-====//
//
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
// See https://llvm.org/LICENSE.txt for license information.
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//

#include "../fp_mode.h"

#define X87_TONEAREST 0x0000
#define X87_DOWNWARD 0x0400
#define X87_UPWARD 0x0800
#define X87_TOWARDZERO 0x0c00
#define X87_RMODE_MASK (X87_TONEAREST | X87_UPWARD | X87_DOWNWARD | X87_TOWARDZERO)

FE_ROUND_MODE __fe_getround() {
// Assume that the rounding mode state for the fpu agrees with the SSE unit.
unsigned short cw;
__asm__ __volatile__ ("fnstcw %0" : "=m" (cw));

switch (cw & X87_RMODE_MASK) {
case X87_TONEAREST:
return FE_TONEAREST;
case X87_DOWNWARD:
return FE_DOWNWARD;
case X87_UPWARD:
return FE_UPWARD;
case X87_TOWARDZERO:
return FE_TOWARDZERO;
}
return FE_TONEAREST;
}

int __fe_raise_inexact() {
float f = 1.0f, g = 3.0f;
__asm__ __volatile__ ("fdivs %1" : "+t" (f) : "m" (g));
return 0;
}
3 changes: 2 additions & 1 deletion compiler-rt/test/builtins/Unit/addtf3_test.c
Expand Up @@ -76,7 +76,8 @@ int main()
UINT64_C(0x61e58dd6c51eb77c)))
return 1;

#if (defined(__arm__) || defined(__aarch64__)) && defined(__ARM_FP)
#if (defined(__arm__) || defined(__aarch64__)) && defined(__ARM_FP) || \
defined(i386) || defined(__x86_64__)
// Rounding mode tests on supported architectures
const long double m = 1234.0L, n = 0.01L;

Expand Down
3 changes: 2 additions & 1 deletion compiler-rt/test/builtins/Unit/subtf3_test.c
Expand Up @@ -69,7 +69,8 @@ int main()
UINT64_C(0xa44a7bca780a166c)))
return 1;

#if (defined(__arm__) || defined(__aarch64__)) && defined(__ARM_FP)
#if (defined(__arm__) || defined(__aarch64__)) && defined(__ARM_FP) || \
defined(i386) || defined(__x86_64__)
// Rounding mode tests on supported architectures
const long double m = 1234.02L, n = 0.01L;

Expand Down

0 comments on commit 1689ad2

Please sign in to comment.