- fix most of the -Wunreachable-code and -Wlogical-op warnings
[libfirm] / ir / tv / fltcalc.c
index 973faf7..893b930 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 1995-2008 University of Karlsruhe.  All right reserved.
+ * Copyright (C) 1995-2010 University of Karlsruhe.  All right reserved.
  *
  * This file is part of libFirm.
  *
  * @author   Mathias Heil
  * @version  $Id$
  */
-
 #include "config.h"
 
 #include "fltcalc.h"
 #include "strcalc.h"
+#include "error.h"
 
-#include <math.h>    /* need isnan() and isinf() (will be changed)*/
+#include <math.h>
 /* undef some reused constants defined by math.h */
 #ifdef NAN
 #  undef NAN
 #endif
 
-#ifdef HAVE_INTTYPES_H
-# include <inttypes.h>
-#endif
+#include <inttypes.h>
 #include <string.h>
 #include <stdlib.h>
 #include <stdio.h>
 
 #include "xmalloc.h"
 
+#if !defined(HAVE_LONG_DOUBLE) || defined(__CYGWIN__)
+/* No strtold on windows and no replacement yet */
+#define strtold(s, e) strtod(s, e)
+#endif
+
 /** The number of extra precision rounding bits */
 #define ROUNDING_BITS 2
 
@@ -137,7 +140,8 @@ static int max_precision;
 static int fc_exact = 1;
 
 #if 0
-static void fail_char(const char *str, unsigned int len, int pos) {
+static void fail_char(const char *str, unsigned int len, int pos)
+{
        if (*(str+pos))
                printf("ERROR: Unexpected character '%c'\n", *(str + pos));
        else
@@ -150,7 +154,8 @@ static void fail_char(const char *str, unsigned int len, int pos) {
 #endif
 
 /** pack machine-like */
-static void *pack(const fp_value *int_float, void *packed) {
+static void *pack(const fp_value *int_float, void *packed)
+{
        char     *shift_val;
        char     *temp;
        fp_value *val_buffer;
@@ -213,7 +218,8 @@ static void *pack(const fp_value *int_float, void *packed) {
  *
  * @return non-zero if result is exact
  */
-static int normalize(const fp_value *in_val, fp_value *out_val, int sticky) {
+static int normalize(const fp_value *in_val, fp_value *out_val, int sticky)
+{
        int exact = 1;
        int hsb;
        char lsb, guard, round, round_dir = 0;
@@ -414,7 +420,8 @@ do {                                                      \
 /**
  * calculate a + b, where a is the value with the bigger exponent
  */
-static void _fadd(const fp_value *a, const fp_value *b, fp_value *result) {
+static void _fadd(const fp_value *a, const fp_value *b, fp_value *result)
+{
        char *temp;
        char *exp_diff;
 
@@ -532,7 +539,8 @@ static void _fadd(const fp_value *a, const fp_value *b, fp_value *result) {
 /**
  * calculate a * b
  */
-static void _fmul(const fp_value *a, const fp_value *b, fp_value *result) {
+static void _fmul(const fp_value *a, const fp_value *b, fp_value *result)
+{
        int sticky;
        char *temp;
        char res_sign;
@@ -618,7 +626,8 @@ static void _fmul(const fp_value *a, const fp_value *b, fp_value *result) {
 /**
  * calculate a / b
  */
-static void _fdiv(const fp_value *a, const fp_value *b, fp_value *result) {
+static void _fdiv(const fp_value *a, const fp_value *b, fp_value *result)
+{
        int sticky;
        char *temp, *dividend;
        char res_sign;
@@ -717,7 +726,8 @@ static void _fdiv(const fp_value *a, const fp_value *b, fp_value *result) {
 }
 
 #if 0
-static void _power_of_ten(int exp, ieee_descriptor_t *desc, char *result) {
+static void _power_of_ten(int exp, ieee_descriptor_t *desc, char *result)
+{
        char *build;
        char *temp;
 
@@ -759,7 +769,8 @@ static void _power_of_ten(int exp, ieee_descriptor_t *desc, char *result) {
  *
  * This does not clip to any integer range.
  */
-static void _trunc(const fp_value *a, fp_value *result) {
+static void _trunc(const fp_value *a, fp_value *result)
+{
        /*
         * When exponent == 0 all bits left of the radix point
         * are the integral part of the value. For 15bit exp_size
@@ -820,223 +831,42 @@ static void _trunc(const fp_value *a, fp_value *result) {
 /********
  * functions defined in fltcalc.h
  ********/
-const void *fc_get_buffer(void) {
+const void *fc_get_buffer(void)
+{
        return calc_buffer;
 }
 
-int fc_get_buffer_length(void) {
+int fc_get_buffer_length(void)
+{
        return calc_buffer_size;
 }
 
-void *fc_val_from_str(const char *str, unsigned int len, const ieee_descriptor_t *desc, void *result) {
-#if 0
-       enum {
-               START,
-               LEFT_OF_DOT,
-               RIGHT_OF_DOT,
-               EXP_START,
-               EXPONENT,
-               END
-       };
-
-       char exp_sign;
-       int exp_int, hsb, state;
-
-       const char *old_str;
-
-       int pos;
-       char *mant_str, *exp_val, *power_val;
-
-       (void) len;
-       if (result == NULL) result = calc_buffer;
-
-       exp_val = alloca(value_size);
-       power_val = alloca(calc_buffer_size);
-       mant_str = alloca((len)?(len):(strlen(str)));
-
-       result->desc.exponent_size = desc->exponent_size;
-       result->desc.mantissa_size = desc->mantissa_size;
-       result->desc.explicit_one  = desc->explicit_one;
-       result->desc.clss          = NORMAL;
-
-       old_str = str;
-       pos = 0;
-       exp_int = 0;
-       state = START;
-
-       while (len == 0 || str-old_str < len) {
-               switch (state) {
-               case START:
-                       switch (*str) {
-                       case '+':
-                               result->sign = 0;
-                               state = LEFT_OF_DOT;
-                               str++;
-                               break;
-
-                       case '-':
-                               result->sign = 1;
-                               state = LEFT_OF_DOT;
-                               str++;
-                               break;
-
-                       case '0': case '1': case '2': case '3': case '4': case '5': case '6': case '7': case '8': case '9':
-                               result->sign = 0;
-                               state = LEFT_OF_DOT;
-                               break;
-
-                       case '.':
-                               result->sign = 0;
-                               state = RIGHT_OF_DOT;
-                               str++;
-                               break;
-
-                       case 'n':
-                       case 'N':
-                       case 'i':
-                       case 'I':
-                               break;
-
-                       default:
-                               fail_char(old_str, len, str - old_str);
-                       }
-                       break;
-
-               case LEFT_OF_DOT:
-                       switch (*str) {
-                       case '0': case '1': case '2': case '3': case '4': case '5': case '6': case '7': case '8': case '9':
-                               mant_str[pos++] = *(str++);
-                               break;
-
-                       case '.':
-                               state = RIGHT_OF_DOT;
-                               str++;
-                               break;
-
-                       case 'e':
-                       case 'E':
-                               state = EXP_START;
-                               str++;
-                               break;
-
-                       case '\0':
-                               mant_str[pos] = '\0';
-                               goto done;
-
-                       default:
-                               fail_char(old_str, len, str - old_str);
-                       }
-                       break;
-
-               case RIGHT_OF_DOT:
-                       switch (*str) {
-                       case '0': case '1': case '2': case '3': case '4': case '5': case '6': case '7': case '8': case '9':
-                               mant_str[pos++] = *(str++);
-                               exp_int++;
-                               break;
-
-                       case 'e':
-                       case 'E':
-                               state = EXP_START;
-                               str++;
-                               break;
-
-                       case '\0':
-                               mant_str[pos] = '\0';
-                               goto done;
-
-                       default:
-                               fail_char(old_str, len, str - old_str);
-                       }
-                       break;
-
-               case EXP_START:
-                       switch (*str) {
-                       case '-':
-                               exp_sign = 1;
-                               /* fall through */
-                       case '+':
-                               if (*(str-1) != 'e' && *(str-1) != 'E') fail_char(old_str, len, str - old_str);
-                               str++;
-                               break;
-
-                       case '0': case '1': case '2': case '3': case '4': case '5': case '6': case '7': case '8': case '9':
-                               mant_str[pos] = '\0';
-                               pos = 1;
-                               str++;
-                               state = EXPONENT;
-                               break;
-
-                       default:
-                               fail_char(old_str, len, str - old_str);
-                       }
-                       break;
-
-               case EXPONENT:
-                       switch (*str) {
-                       case '0': case '1': case '2': case '3': case '4': case '5': case '6': case '7': case '8': case '9':
-                               pos++;
-                               str++;
-                               break;
-
-                       case '\0': goto done;
-
-                       default:
-                               fail_char(old_str, len, str - old_str);
-                       }
-               }
-       } /*  switch(state) */
-
-done:
-       sc_val_from_str(mant_str, strlen(mant_str), _mant(result));
-
-       /* shift to put value left of radix point */
-       sc_val_from_ulong(mant_size + ROUNDING_BITS, exp_val);
-
-       _shift_left(_mant(result), exp_val, _mant(result));
-
-       sc_val_from_ulong((1 << (exp_size - 1)) - 1, _exp(result));
-
-       _normalize(result, result, 0);
+void *fc_val_from_str(const char *str, unsigned int len, const ieee_descriptor_t *desc, void *result)
+{
+       char *buffer;
 
-       if (state == EXPONENT) {
-               exp_int -= atoi(str-pos);
-       }
-
-       _power_of_ten(exp_int, &result->desc, power_val);
-
-       _fdiv(result, power_val, result);
-
-       return result;
-#else
        /* XXX excuse of an implementation to make things work */
        LLDBL             val;
        fp_value          *tmp = alloca(calc_buffer_size);
        ieee_descriptor_t tmp_desc;
-       (void) len;
 
-#if defined(HAVE_LONG_DOUBLE) && !defined(__CYGWIN__)
-       val = strtold(str, NULL);
+       buffer = alloca(len+1);
+       memcpy(buffer, str, len);
+       buffer[len] = '\0';
+       val = strtold(buffer, NULL);
+
        DEBUGPRINTF(("val_from_str(%s)\n", str));
        tmp_desc.exponent_size = 15;
        tmp_desc.mantissa_size = 63;
        tmp_desc.explicit_one  = 1;
        tmp_desc.clss          = NORMAL;
        fc_val_from_ieee754(val, &tmp_desc, tmp);
-#else
-       val = strtod(str, NULL);
-       DEBUGPRINTF(("val_from_str(%s)\n", str));
-       tmp_desc.exponent_size = 11;
-       tmp_desc.mantissa_size = 52;
-       tmp_desc.explicit_one  = 0;
-       tmp_desc.clss          = NORMAL;
-       fc_val_from_ieee754(val, &tmp_desc, tmp);
-#endif /* HAVE_LONG_DOUBLE */
+
        return fc_cast(tmp, desc, result);
-#endif
 }
 
-fp_value *fc_val_from_ieee754(LLDBL l, const ieee_descriptor_t *desc, fp_value *result) {
+fp_value *fc_val_from_ieee754(LLDBL l, const ieee_descriptor_t *desc, fp_value *result)
+{
        char    *temp;
        int     bias_res, bias_val, mant_val;
        value_t srcval;
@@ -1141,7 +971,8 @@ fp_value *fc_val_from_ieee754(LLDBL l, const ieee_descriptor_t *desc, fp_value *
        return result;
 }
 
-LLDBL fc_val_to_ieee754(const fp_value *val) {
+LLDBL fc_val_to_ieee754(const fp_value *val)
+{
        fp_value *value;
        fp_value *temp = NULL;
 
@@ -1207,7 +1038,8 @@ LLDBL fc_val_to_ieee754(const fp_value *val) {
        return buildval.d;
 }
 
-fp_value *fc_cast(const fp_value *value, const ieee_descriptor_t *desc, fp_value *result) {
+fp_value *fc_cast(const fp_value *value, const ieee_descriptor_t *desc, fp_value *result)
+{
        char *temp;
        int exp_offset, val_bias, res_bias;
 
@@ -1228,6 +1060,12 @@ fp_value *fc_cast(const fp_value *value, const ieee_descriptor_t *desc, fp_value
                else
                        return fc_get_snan(desc, result);
        }
+       else if (value->desc.clss == INF) {
+               if (value->sign == 0)
+                       return fc_get_plusinf(desc, result);
+               else
+                       return fc_get_minusinf(desc, result);
+       }
 
        /* set the descriptor of the new value */
        result->desc.exponent_size = desc->exponent_size;
@@ -1262,7 +1100,8 @@ fp_value *fc_cast(const fp_value *value, const ieee_descriptor_t *desc, fp_value
        return result;
 }
 
-fp_value *fc_get_max(const ieee_descriptor_t *desc, fp_value *result) {
+fp_value *fc_get_max(const ieee_descriptor_t *desc, fp_value *result)
+{
        if (result == NULL) result = calc_buffer;
 
        result->desc.exponent_size = desc->exponent_size;
@@ -1281,7 +1120,8 @@ fp_value *fc_get_max(const ieee_descriptor_t *desc, fp_value *result) {
        return result;
 }
 
-fp_value *fc_get_min(const ieee_descriptor_t *desc, fp_value *result) {
+fp_value *fc_get_min(const ieee_descriptor_t *desc, fp_value *result)
+{
        if (result == NULL) result = calc_buffer;
 
        fc_get_max(desc, result);
@@ -1290,7 +1130,8 @@ fp_value *fc_get_min(const ieee_descriptor_t *desc, fp_value *result) {
        return result;
 }
 
-fp_value *fc_get_snan(const ieee_descriptor_t *desc, fp_value *result) {
+fp_value *fc_get_snan(const ieee_descriptor_t *desc, fp_value *result)
+{
        if (result == NULL) result = calc_buffer;
 
        result->desc.exponent_size = desc->exponent_size;
@@ -1308,7 +1149,8 @@ fp_value *fc_get_snan(const ieee_descriptor_t *desc, fp_value *result) {
        return result;
 }
 
-fp_value *fc_get_qnan(const ieee_descriptor_t *desc, fp_value *result) {
+fp_value *fc_get_qnan(const ieee_descriptor_t *desc, fp_value *result)
+{
        if (result == NULL) result = calc_buffer;
 
        result->desc.exponent_size = desc->exponent_size;
@@ -1329,7 +1171,10 @@ fp_value *fc_get_qnan(const ieee_descriptor_t *desc, fp_value *result) {
        return result;
 }
 
-fp_value *fc_get_plusinf(const ieee_descriptor_t *desc, fp_value *result) {
+fp_value *fc_get_plusinf(const ieee_descriptor_t *desc, fp_value *result)
+{
+       char *mant;
+
        if (result == NULL) result = calc_buffer;
 
        result->desc.exponent_size = desc->exponent_size;
@@ -1341,12 +1186,17 @@ fp_value *fc_get_plusinf(const ieee_descriptor_t *desc, fp_value *result) {
 
        sc_val_from_ulong((1 << desc->exponent_size) - 1, _exp(result));
 
-       sc_val_from_ulong(0, _mant(result));
+       mant = _mant(result);
+       sc_val_from_ulong(0, mant);
+       if (desc->explicit_one) {
+               sc_set_bit_at(mant, result->desc.mantissa_size + ROUNDING_BITS);
+       }
 
        return result;
 }
 
-fp_value *fc_get_minusinf(const ieee_descriptor_t *desc, fp_value *result) {
+fp_value *fc_get_minusinf(const ieee_descriptor_t *desc, fp_value *result)
+{
        if (result == NULL) result = calc_buffer;
 
        fc_get_plusinf(desc, result);
@@ -1355,7 +1205,8 @@ fp_value *fc_get_minusinf(const ieee_descriptor_t *desc, fp_value *result) {
        return result;
 }
 
-int fc_comp(const fp_value *val_a, const fp_value *val_b) {
+int fc_comp(const fp_value *val_a, const fp_value *val_b)
+{
        int mul = 1;
 
        /*
@@ -1402,27 +1253,33 @@ int fc_comp(const fp_value *val_a, const fp_value *val_b) {
        }
 }
 
-int fc_is_zero(const fp_value *a) {
+int fc_is_zero(const fp_value *a)
+{
        return a->desc.clss == ZERO;
 }
 
-int fc_is_negative(const fp_value *a) {
+int fc_is_negative(const fp_value *a)
+{
        return a->sign;
 }
 
-int fc_is_inf(const fp_value *a) {
+int fc_is_inf(const fp_value *a)
+{
        return a->desc.clss == INF;
 }
 
-int fc_is_nan(const fp_value *a) {
+int fc_is_nan(const fp_value *a)
+{
        return a->desc.clss == NAN;
 }
 
-int fc_is_subnormal(const fp_value *a) {
+int fc_is_subnormal(const fp_value *a)
+{
        return a->desc.clss == SUBNORMAL;
 }
 
-char *fc_print(const fp_value *val, char *buf, int buflen, unsigned base) {
+char *fc_print(const fp_value *val, char *buf, int buflen, unsigned base)
+{
        char *mul_1;
        LLDBL flt_val;
 
@@ -1481,7 +1338,8 @@ char *fc_print(const fp_value *val, char *buf, int buflen, unsigned base) {
        return buf;
 }
 
-unsigned char fc_sub_bits(const fp_value *value, unsigned num_bits, unsigned byte_ofs) {
+unsigned char fc_sub_bits(const fp_value *value, unsigned num_bits, unsigned byte_ofs)
+{
        /* this is used to cache the packed version of the value */
        static char *packed_value = NULL;
 
@@ -1494,18 +1352,21 @@ unsigned char fc_sub_bits(const fp_value *value, unsigned num_bits, unsigned byt
 }
 
 /* Returns non-zero if the mantissa is zero, i.e. 1.0Exxx */
-int fc_zero_mantissa(const fp_value *value) {
+int fc_zero_mantissa(const fp_value *value)
+{
        return sc_get_lowest_set_bit(_mant(value)) == ROUNDING_BITS + value->desc.mantissa_size;
 }
 
 /* Returns the exponent of a value. */
-int fc_get_exponent(const fp_value *value) {
+int fc_get_exponent(const fp_value *value)
+{
        int exp_bias = (1 << (value->desc.exponent_size - 1)) - 1;
        return sc_val_to_long(_exp(value)) - exp_bias;
 }
 
 /* Return non-zero if a given value can be converted lossless into another precision */
-int fc_can_lossless_conv_to(const fp_value *value, const ieee_descriptor_t *desc) {
+int fc_can_lossless_conv_to(const fp_value *value, const ieee_descriptor_t *desc)
+{
        int v;
        int exp_bias;
 
@@ -1525,24 +1386,27 @@ int fc_can_lossless_conv_to(const fp_value *value, const ieee_descriptor_t *desc
        if (0 < v && v < (1 << desc->exponent_size) - 1) {
                /* exponent can be encoded, now check the mantissa */
                v = value->desc.mantissa_size + ROUNDING_BITS - sc_get_lowest_set_bit(_mant(value));
-               return v < desc->mantissa_size;
+               return v <= desc->mantissa_size;
        }
        return 0;
 }
 
 
-fc_rounding_mode_t fc_set_rounding_mode(fc_rounding_mode_t mode) {
+fc_rounding_mode_t fc_set_rounding_mode(fc_rounding_mode_t mode)
+{
        if (mode == FC_TONEAREST || mode == FC_TOPOSITIVE || mode == FC_TONEGATIVE || mode == FC_TOZERO)
                rounding_mode = mode;
 
        return rounding_mode;
 }
 
-fc_rounding_mode_t fc_get_rounding_mode(void) {
+fc_rounding_mode_t fc_get_rounding_mode(void)
+{
        return rounding_mode;
 }
 
-void init_fltcalc(int precision) {
+void init_fltcalc(int precision)
+{
        if (calc_buffer == NULL) {
                /* does nothing if already init */
                if (precision == 0) precision = FC_DEFAULT_PRECISION;
@@ -1575,7 +1439,8 @@ void init_fltcalc(int precision) {
        }
 }
 
-void finish_fltcalc (void) {
+void finish_fltcalc (void)
+{
        free(calc_buffer); calc_buffer = NULL;
 }
 
@@ -1584,7 +1449,8 @@ static char buffer[100];
 #endif
 
 /* definition of interface functions */
-fp_value *fc_add(const fp_value *a, const fp_value *b, fp_value *result) {
+fp_value *fc_add(const fp_value *a, const fp_value *b, fp_value *result)
+{
        if (result == NULL) result = calc_buffer;
 
        TRACEPRINTF(("%s ", fc_print(a, buffer, sizeof(buffer), FC_PACKED)));
@@ -1600,7 +1466,8 @@ fp_value *fc_add(const fp_value *a, const fp_value *b, fp_value *result) {
        return result;
 }
 
-fp_value *fc_sub(const fp_value *a, const fp_value *b, fp_value *result) {
+fp_value *fc_sub(const fp_value *a, const fp_value *b, fp_value *result)
+{
        fp_value *temp;
 
        if (result == NULL) result = calc_buffer;
@@ -1620,7 +1487,8 @@ fp_value *fc_sub(const fp_value *a, const fp_value *b, fp_value *result) {
        return result;
 }
 
-fp_value *fc_mul(const fp_value *a, const fp_value *b, fp_value *result) {
+fp_value *fc_mul(const fp_value *a, const fp_value *b, fp_value *result)
+{
        if (result == NULL) result = calc_buffer;
 
        TRACEPRINTF(("%s ", fc_print(a, buffer, sizeof(buffer), FC_PACKED)));
@@ -1632,7 +1500,8 @@ fp_value *fc_mul(const fp_value *a, const fp_value *b, fp_value *result) {
        return result;
 }
 
-fp_value *fc_div(const fp_value *a, const fp_value *b, fp_value *result) {
+fp_value *fc_div(const fp_value *a, const fp_value *b, fp_value *result)
+{
        if (result == NULL) result = calc_buffer;
 
        TRACEPRINTF(("%s ", fc_print(a, buffer, sizeof(buffer), FC_PACKED)));
@@ -1644,7 +1513,8 @@ fp_value *fc_div(const fp_value *a, const fp_value *b, fp_value *result) {
        return result;
 }
 
-fp_value *fc_neg(const fp_value *a, fp_value *result) {
+fp_value *fc_neg(const fp_value *a, fp_value *result)
+{
        if (result == NULL) result = calc_buffer;
 
        TRACEPRINTF(("- %s ", fc_print(a, buffer, sizeof(buffer), FC_PACKED)));
@@ -1657,7 +1527,8 @@ fp_value *fc_neg(const fp_value *a, fp_value *result) {
        return result;
 }
 
-fp_value *fc_int(const fp_value *a, fp_value *result) {
+fp_value *fc_int(const fp_value *a, fp_value *result)
+{
        if (result == NULL) result = calc_buffer;
 
        TRACEPRINTF(("%s ", fc_print(a, buffer, sizeof(buffer), FC_PACKED)));
@@ -1669,43 +1540,48 @@ fp_value *fc_int(const fp_value *a, fp_value *result) {
        return result;
 }
 
-fp_value *fc_rnd(const fp_value *a, fp_value *result) {
+fp_value *fc_rnd(const fp_value *a, fp_value *result)
+{
        if (result == NULL) result = calc_buffer;
 
        (void) a;
        TRACEPRINTF(("%s ", fc_print(a, buffer, sizeof(buffer), FC_PACKED)));
        TRACEPRINTF(("rounded to integer "));
 
-       assert(!"fc_rnd() not yet implemented");
-
-       TRACEPRINTF(("= %s\n", fc_print(result, buffer, sizeof(buffer), FC_PACKED)));
-       return result;
+       panic("fc_rnd() not yet implemented");
 }
 
 /*
  * convert a floating point value into an sc value ...
  */
-int fc_flt2int(const fp_value *a, void *result, ir_mode *dst_mode) {
+int fc_flt2int(const fp_value *a, void *result, ir_mode *dst_mode)
+{
        if (a->desc.clss == NORMAL) {
                int exp_bias = (1 << (a->desc.exponent_size - 1)) - 1;
                int exp_val  = sc_val_to_long(_exp(a)) - exp_bias;
                int shift, highest;
                int mantissa_size;
+               int tgt_bits;
 
                if (a->sign && !mode_is_signed(dst_mode)) {
                        /* FIXME: for now we cannot convert this */
                        return 0;
                }
 
+               tgt_bits = get_mode_size_bits(dst_mode);
+               if (mode_is_signed(dst_mode))
+                       --tgt_bits;
+
                assert(exp_val >= 0 && "floating point value not integral before fc_flt2int() call");
                mantissa_size = a->desc.mantissa_size + ROUNDING_BITS;
                shift         = exp_val - mantissa_size;
 
-               mantissa_size += a->desc.explicit_one;
+               if (tgt_bits < mantissa_size + 1)
+                       tgt_bits = mantissa_size + 1;
                if (shift > 0) {
-                       sc_shlI(_mant(a),  shift, mantissa_size, 0, result);
+                       sc_shlI(_mant(a),  shift, tgt_bits, 0, result);
                } else {
-                       sc_shrI(_mant(a), -shift, mantissa_size, 0, result);
+                       sc_shrI(_mant(a), -shift, tgt_bits, 0, result);
                }
 
                /* check for overflow */
@@ -1744,13 +1620,15 @@ int fc_flt2int(const fp_value *a, void *result, ir_mode *dst_mode) {
 }
 
 
-unsigned fc_set_immediate_precision(unsigned bits) {
+unsigned fc_set_immediate_precision(unsigned bits)
+{
        unsigned old = immediate_prec;
 
        immediate_prec = bits;
        return old;
 }
 
-int fc_is_exact(void) {
+int fc_is_exact(void)
+{
        return fc_exact;
 }