remove #ifdef HAVE_CONFIG_Hs
[libfirm] / ir / opt / opt_confirms.c
index 2e428de..3fd45f7 100644 (file)
@@ -1,22 +1,34 @@
 /*
- * Project:     libFIRM
- * File name:   ir/opt/opt_confirms.c
- * Purpose:     Optimizations regarding Confirm nodes
- * Author:      Michael Beck
- * Modified by:
- * Created:
- * CVS-ID:      $Id$
- * Copyright:   (c) 1998-2005 Universität Karlsruhe
- * Licence:     This file protected by GPL -  GNU GENERAL PUBLIC LICENSE.
+ * Copyright (C) 1995-2008 University of Karlsruhe.  All right reserved.
+ *
+ * This file is part of libFirm.
+ *
+ * This file may be distributed and/or modified under the terms of the
+ * GNU General Public License version 2 as published by the Free Software
+ * Foundation and appearing in the file LICENSE.GPL included in the
+ * packaging of this file.
+ *
+ * Licensees holding valid libFirm Professional Edition licenses may use
+ * this file in accordance with the libFirm Commercial License.
+ * Agreement provided with the Software.
+ *
+ * This file is provided AS IS with NO WARRANTY OF ANY KIND, INCLUDING THE
+ * WARRANTY OF DESIGN, MERCHANTABILITY AND FITNESS FOR A PARTICULAR
+ * PURPOSE.
  */
 
-#ifdef HAVE_CONFIG_H
-# include "config.h"
-#endif
+/**
+ * @file
+ * @brief   Optimizations regarding Confirm nodes.
+ * @author  Michael Beck
+ * @version $Id$
+ */
+#include "config.h"
 
 #undef DEBUG_CONFIRM
 
 #include "tv_t.h"
+#include "irnode_t.h"
 #include "iropt_t.h"
 #include "iropt_dbg.h"
 #include "opt_confirms.h"
 #include "irprintf.h"
 
 enum range_tags {
-  MIN_INCLUDED = 0x00,  /**< [min, ... */
-  MAX_INCLUDED = 0x00,  /**< ..., max] */
-  MIN_EXCLUDED = 0x01,  /**< (min, ... */
-  MAX_EXCLUDED = 0x02   /**< ..., max) */
+       MIN_INCLUDED = 0x00,  /**< [min, ... */
+       MAX_INCLUDED = 0x00,  /**< ..., max] */
+       MIN_EXCLUDED = 0x01,  /**< (min, ... */
+       MAX_EXCLUDED = 0x02   /**< ..., max) */
 };
 
 /**
@@ -39,9 +51,9 @@ enum range_tags {
  * all intervals.
  */
 typedef struct _interval_t {
-  tarval        *min;   /**< lowest border */
-  tarval        *max;   /**< highest border */
-  unsigned char flags;  /**< border flags */
+       tarval        *min;   /**< lowest border */
+       tarval        *max;   /**< highest border */
+       unsigned char flags;  /**< border flags */
 } interval_t;
 
 #ifdef DEBUG_CONFIRM
@@ -83,64 +95,71 @@ static tarval *compare_iv_dbg(const interval_t *l_iv, const interval_t *r_iv, pn
  * This is a often needed case, so we handle here Confirm
  * nodes too.
  */
-int value_not_zero(ir_node *n, ir_node **confirm)
-{
+int value_not_zero(const ir_node *n, ir_node_cnst_ptr *confirm) {
 #define RET_ON(x)  if (x) { *confirm = n; return 1; }; break
 
-  tarval *tv;
-  ir_mode *mode = get_irn_mode(n);
-  pn_Cmp pnc;
-
-  *confirm = NULL;
-  while (get_irn_op(n) == op_Confirm) {
-    /*
-     * Note: A Confirm is never after a Const. So,
-     * we simply can check the bound for being a Const
-     * without the fear that is might be hidden by a further Confirm.
-     */
-    tv = value_of(get_Confirm_bound(n));
-    if (tv == tarval_bad)
-      return 0;
-
-    pnc = tarval_cmp(tv, get_mode_null(mode));
-
-    /*
-     * Beware: C might by a NaN. It is not clear, what we should do
-     * than. Of course a NaN is != 0, but we might use this function
-     * to remove up Exceptions, and NaN's might generate Exception.
-     * So, we do NOT handle NaNs here for safety.
-     *
-     * Note that only the C != 0 case need additional checking.
-     */
-    switch (get_Confirm_cmp(n)) {
-    case pn_Cmp_Eq: /* n == C /\ C != 0 ==> n != 0 */
-      RET_ON(pnc != pn_Cmp_Eq && pnc != pn_Cmp_Uo);
-    case pn_Cmp_Lg: /* n != C /\ C == 0 ==> n != 0 */
-      RET_ON(pnc == pn_Cmp_Eq);
-    case pn_Cmp_Lt: /* n <  C /\ C <= 0 ==> n != 0 */
-      RET_ON(pnc == pn_Cmp_Lt || pnc == pn_Cmp_Eq);
-    case pn_Cmp_Le: /* n <= C /\ C <  0 ==> n != 0 */
-      RET_ON(pnc == pn_Cmp_Lt);
-    case pn_Cmp_Ge: /* n >= C /\ C >  0 ==> n != 0 */
-      RET_ON(pnc == pn_Cmp_Gt);
-    case pn_Cmp_Gt: /* n >  C /\ C >= 0 ==> n != 0 */
-      RET_ON(pnc == pn_Cmp_Gt || pnc == pn_Cmp_Eq);
-    default:
-      break;
-    }
-
-    /* there might be several Confirms one after other that form an interval */
-    n = get_Confirm_value(n);
-  }
-  tv = value_of(n);
-
-  if (tv == tarval_bad)
-    return 0;
-
-  pnc = tarval_cmp(tv, get_mode_null(mode));
-
-  /* again, need check for NaN */
-  return (pnc != pn_Cmp_Eq) && (pnc != pn_Cmp_Uo);
+       tarval *tv;
+       ir_mode *mode = get_irn_mode(n);
+       pn_Cmp pnc;
+
+       *confirm = NULL;
+
+       /* there might be several Confirms one after other that form an interval */
+       for (;;) {
+               if (is_Minus(n) || is_Abs(n)) {
+                       /* we can safely skip Minus and Abs when checking for != 0 */
+                       n = get_unop_op(n);
+                       continue;
+               }
+               if (! is_Confirm(n))
+                       break;
+
+               /*
+                * Note: A Confirm is never after a Const. So,
+                * we simply can check the bound for being a Const
+                * without the fear that is might be hidden by a further Confirm.
+                */
+               tv = value_of(get_Confirm_bound(n));
+               if (tv == tarval_bad)
+                       return 0;
+
+               pnc = tarval_cmp(tv, get_mode_null(mode));
+
+               /*
+                * Beware: C might by a NaN. It is not clear, what we should do
+                * than. Of course a NaN is != 0, but we might use this function
+                * to remove up Exceptions, and NaN's might generate Exception.
+                * So, we do NOT handle NaNs here for safety.
+                *
+                * Note that only the C != 0 case need additional checking.
+                */
+               switch (get_Confirm_cmp(n)) {
+               case pn_Cmp_Eq: /* n == C /\ C != 0 ==> n != 0 */
+                       RET_ON(pnc != pn_Cmp_Eq && pnc != pn_Cmp_Uo);
+               case pn_Cmp_Lg: /* n != C /\ C == 0 ==> n != 0 */
+                       RET_ON(pnc == pn_Cmp_Eq);
+               case pn_Cmp_Lt: /* n <  C /\ C <= 0 ==> n != 0 */
+                       RET_ON(pnc == pn_Cmp_Lt || pnc == pn_Cmp_Eq);
+               case pn_Cmp_Le: /* n <= C /\ C <  0 ==> n != 0 */
+                       RET_ON(pnc == pn_Cmp_Lt);
+               case pn_Cmp_Ge: /* n >= C /\ C >  0 ==> n != 0 */
+                       RET_ON(pnc == pn_Cmp_Gt);
+               case pn_Cmp_Gt: /* n >  C /\ C >= 0 ==> n != 0 */
+                       RET_ON(pnc == pn_Cmp_Gt || pnc == pn_Cmp_Eq);
+               default:
+                       break;
+               }
+               n = get_Confirm_value(n);
+       }
+       tv = value_of(n);
+
+       if (tv == tarval_bad)
+               return 0;
+
+       pnc = tarval_cmp(tv, get_mode_null(mode));
+
+       /* again, need check for NaN */
+       return (pnc != pn_Cmp_Eq) && (pnc != pn_Cmp_Uo);
 
 #undef RET_ON
 }  /* value_not_zero */
@@ -154,37 +173,44 @@ int value_not_zero(ir_node *n, ir_node **confirm)
  * - A SymConst(entity) is NEVER a NULL pointer
  * - Confirms are evaluated
  */
-int value_not_null(ir_node *n, ir_node **confirm)
-{
-  ir_op *op;
-
-  *confirm = NULL;
-  n  = skip_Cast(n);
-  op = get_irn_op(n);
-  assert(mode_is_reference(get_irn_mode(n)));
-  if (get_opt_sel_based_null_check_elim()) {
-    /* skip all Sel nodes and Cast's */
-    while (op == op_Sel) {
-      n = skip_Cast(get_Sel_ptr(n));
-      op = get_irn_op(n);
-    }
-  }
-  if (op == op_SymConst && get_SymConst_kind(n) == symconst_addr_ent)
-    return 1;
-  if (op == op_Const) {
-    tarval *tv = get_Const_tarval(n);
-
-    if (tv != tarval_bad && classify_tarval(tv) != TV_CLASSIFY_NULL)
-      return 1;
-  }
-  else if (op == op_Confirm) {
-    if (get_Confirm_cmp(n) == pn_Cmp_Lg &&
-        classify_Const(get_Confirm_bound(n)) == CNST_NULL) {
-      *confirm = n;
-      return 1;
-    }
-  }
-  return 0;
+int value_not_null(const ir_node *n, ir_node_cnst_ptr *confirm) {
+       tarval *tv;
+
+       *confirm = NULL;
+       n  = skip_Cast_const(n);
+
+       tv = value_of(n);
+       if (tarval_is_constant(tv) && ! tarval_is_null(tv))
+               return 1;
+
+       assert(mode_is_reference(get_irn_mode(n)));
+       if (get_opt_sel_based_null_check_elim()) {
+               /* skip all Sel nodes and Cast's */
+               while (is_Sel(n)) {
+                       n = skip_Cast(get_Sel_ptr(n));
+               }
+       }
+       if (is_Global(n)) {
+               /* global references are never NULL */
+               return 1;
+       } else if (n == get_irg_frame(current_ir_graph)) {
+               /* local references are never NULL */
+               return 1;
+       } else {
+               /* check for more Confirms */
+               for (; is_Confirm(n); n = skip_Cast(get_Confirm_value(n))) {
+                       if (get_Confirm_cmp(n) == pn_Cmp_Lg) {
+                               ir_node *bound = get_Confirm_bound(n);
+                               tarval  *tv    = value_of(bound);
+
+                               if (tarval_is_null(tv)) {
+                                       *confirm = n;
+                                       return 1;
+                               }
+                       }
+               }
+       }
+       return 0;
 }  /* value_not_null */
 
 /*
@@ -192,91 +218,105 @@ int value_not_null(ir_node *n, ir_node **confirm)
  * If the mode of the value did not honor signed zeros, else
  * check for >= 0 or < 0.
  */
-value_classify_sign classify_value_sign(ir_node *n)
-{
-  tarval *tv, *c;
-  ir_mode *mode;
-  pn_Cmp cmp, ncmp;
-
-  if (get_irn_op(n) != op_Confirm)
-    return value_classified_unknown;
-
-  tv  = value_of(get_Confirm_bound(n));
-  if (tv == tarval_bad)
-    return value_classified_unknown;
-
-  mode = get_irn_mode(n);
-
-  /*
-   * We can handle only >=, >, <, <= cases.
-   * We could handle == too, but this will be optimized into
-   * a constant either.
-   *
-   * Note that for integer modes we have a slightly better
-   * optimization possibilities, so we handle this
-   * different.
-   */
-  cmp = get_Confirm_cmp(n);
-
-  switch (cmp) {
-  case pn_Cmp_Lt:
-    /*
-     * must be x < c <= 1 to be useful if integer mode and -0 = 0
-     *         x < c <= 0 to be useful else
-     */
-  case pn_Cmp_Le:
-    /*
-     * must be x <= c < 1 to be useful if integer mode and -0 = 0
-     *         x <= c < 0 to be useful else
-     */
-    c = mode_is_int(mode) && mode_honor_signed_zeros(mode) ?
-      get_mode_one(mode) : get_mode_null(mode);
-
-    ncmp = tarval_cmp(tv, c);
-    if (ncmp == pn_Cmp_Eq)
-      ncmp = pn_Cmp_Le;
-
-    if (cmp != (ncmp ^ pn_Cmp_Eq))
-      return value_classified_unknown;
-
-    /* yep, negative */
-    return value_classified_negative;
-
-  case pn_Cmp_Ge:
-    /*
-     * must be x >= c > -1 to be useful if integer mode
-     *         x >= c >= 0 to be useful else
-     */
-  case pn_Cmp_Gt:
-    /*
-     * must be x > c >= -1 to be useful if integer mode
-     *         x > c >= 0 to be useful else
-     */
-    if (mode_is_int(mode)) {
-      c = get_mode_minus_one(mode);
-
-      ncmp = tarval_cmp(tv, c);
-      if (ncmp == pn_Cmp_Eq)
-        ncmp = pn_Cmp_Ge;
-
-      if (cmp != (ncmp ^ pn_Cmp_Eq))
-        return value_classified_unknown;
-    }
-    else {
-      c = get_mode_minus_one(mode);
-
-      ncmp = tarval_cmp(tv, c);
-
-      if (ncmp != pn_Cmp_Eq && ncmp != pn_Cmp_Gt)
-        return value_classified_unknown;
-    }
-
-    /* yep, positive */
-    return value_classified_positive;
-
-  default:
-    return value_classified_unknown;
-  }
+value_classify_sign classify_value_sign(ir_node *n) {
+       tarval *tv, *c;
+       ir_mode *mode;
+       pn_Cmp cmp, ncmp;
+       int negate = 1;
+
+       for (;;) {
+               ir_opcode code = get_irn_opcode(n);
+
+               switch (code) {
+               case iro_Minus:
+                       negate *= -1;
+                       n = get_Minus_op(n);
+                       continue;
+               case iro_Confirm:
+                       break;
+               default:
+                       return value_classified_unknown;
+               }
+               break;
+       }
+       if (!is_Confirm(n))
+               return value_classified_unknown;
+
+       tv  = value_of(get_Confirm_bound(n));
+       if (tv == tarval_bad)
+               return value_classified_unknown;
+
+       mode = get_irn_mode(n);
+
+       /*
+        * We can handle only >=, >, <, <= cases.
+        * We could handle == too, but this will be optimized into
+        * a constant either.
+        *
+        * Note that for integer modes we have a slightly better
+        * optimization possibilities, so we handle this
+        * different.
+        */
+       cmp = get_Confirm_cmp(n);
+
+       switch (cmp) {
+       case pn_Cmp_Lt:
+               /*
+                * must be x < c <= 1 to be useful if integer mode and -0 = 0
+                *         x < c <= 0 to be useful else
+                */
+       case pn_Cmp_Le:
+               /*
+                * must be x <= c < 1 to be useful if integer mode and -0 = 0
+                *         x <= c < 0 to be useful else
+                */
+               c = mode_is_int(mode) && mode_honor_signed_zeros(mode) ?
+                       get_mode_one(mode) : get_mode_null(mode);
+
+               ncmp = tarval_cmp(tv, c);
+               if (ncmp == pn_Cmp_Eq)
+                       ncmp = pn_Cmp_Le;
+
+               if (cmp != (ncmp ^ pn_Cmp_Eq))
+                       return value_classified_unknown;
+
+               /* yep, negative */
+               return value_classified_negative * negate;
+
+       case pn_Cmp_Ge:
+               /*
+                * must be x >= c > -1 to be useful if integer mode
+                *         x >= c >= 0 to be useful else
+                */
+       case pn_Cmp_Gt:
+               /*
+                * must be x > c >= -1 to be useful if integer mode
+                *         x > c >= 0 to be useful else
+                */
+               if (mode_is_int(mode)) {
+                       c = get_mode_minus_one(mode);
+
+                       ncmp = tarval_cmp(tv, c);
+                       if (ncmp == pn_Cmp_Eq)
+                               ncmp = pn_Cmp_Ge;
+
+                       if (cmp != (ncmp ^ pn_Cmp_Eq))
+                               return value_classified_unknown;
+               } else {
+                       c = get_mode_minus_one(mode);
+
+                       ncmp = tarval_cmp(tv, c);
+
+                       if (ncmp != pn_Cmp_Eq && ncmp != pn_Cmp_Gt)
+                               return value_classified_unknown;
+               }
+
+               /* yep, positive */
+               return value_classified_positive * negate;
+
+       default:
+               return value_classified_unknown;
+       }
 }  /* classify_value_sign */
 
 /**
@@ -285,43 +325,41 @@ value_classify_sign classify_value_sign(ir_node *n)
  * @return the filled interval or NULL if no interval
  *         can be created (happens only on floating point
  */
-static interval_t *get_interval_from_tv(interval_t *iv, tarval *tv)
-{
-  ir_mode *mode = get_tarval_mode(tv);
-
-  if (tv == tarval_bad) {
-    if (mode_is_float(mode)) {
-      /* NaN could be included which we cannot handle */
-      iv->min   = tarval_bad;
-      iv->max   = tarval_bad;
-      iv->flags = MIN_EXCLUDED | MAX_EXCLUDED;
-      return NULL;
-    }
-    else {
-      /* [-oo, +oo] */
-      iv->min   = get_mode_min(mode);
-      iv->max   = get_mode_max(mode);
-      iv->flags = MIN_INCLUDED | MAX_INCLUDED;
-      return iv;
-    }
-  }
-
-  if (mode_is_float(mode)) {
-    if (tv == get_mode_NAN(mode)) {
-      /* arg, we cannot handle NaN's. */
-      iv->min   = tarval_bad;
-      iv->max   = tarval_bad;
-      iv->flags = MIN_EXCLUDED | MAX_EXCLUDED;
-      return NULL;
-    }
-  }
-
-  /* [tv, tv] */
-  iv->min   = tv;
-  iv->max   = tv;
-  iv->flags = MIN_INCLUDED | MAX_INCLUDED;
-
-  return iv;
+static interval_t *get_interval_from_tv(interval_t *iv, tarval *tv) {
+       ir_mode *mode = get_tarval_mode(tv);
+
+       if (tv == tarval_bad) {
+               if (mode_is_float(mode)) {
+                       /* NaN could be included which we cannot handle */
+                       iv->min   = tarval_bad;
+                       iv->max   = tarval_bad;
+                       iv->flags = MIN_EXCLUDED | MAX_EXCLUDED;
+                       return NULL;
+               } else {
+                       /* [-oo, +oo] */
+                       iv->min   = get_mode_min(mode);
+                       iv->max   = get_mode_max(mode);
+                       iv->flags = MIN_INCLUDED | MAX_INCLUDED;
+                       return iv;
+               }
+       }
+
+       if (mode_is_float(mode)) {
+               if (tv == get_mode_NAN(mode)) {
+                       /* arg, we cannot handle NaN's. */
+                       iv->min   = tarval_bad;
+                       iv->max   = tarval_bad;
+                       iv->flags = MIN_EXCLUDED | MAX_EXCLUDED;
+                       return NULL;
+               }
+       }
+
+       /* [tv, tv] */
+       iv->min   = tv;
+       iv->max   = tv;
+       iv->flags = MIN_INCLUDED | MAX_INCLUDED;
+
+       return iv;
 }  /* get_interval_from_tv */
 
 /**
@@ -334,96 +372,95 @@ static interval_t *get_interval_from_tv(interval_t *iv, tarval *tv)
  * @return the filled interval or NULL if no interval
  *         can be created (happens only on floating point
  */
-static interval_t *get_interval(interval_t *iv, ir_node *bound, pn_Cmp pnc)
-{
-  ir_mode *mode = get_irn_mode(bound);
-  tarval  *tv   = value_of(bound);
-
-  if (tv == tarval_bad) {
-    /* There is nothing we could do here. For integer
-     * modes we could return [-oo, +oo], but there is
-     * nothing we could deduct from such an interval.
-     * So, speed things up and return unknown.
-     */
-    iv->min   = tarval_bad;
-    iv->max   = tarval_bad;
-    iv->flags = MIN_EXCLUDED | MAX_EXCLUDED;
-    return NULL;
-  }
-
-  if (mode_is_float(mode)) {
-    if (tv == get_mode_NAN(mode)) {
-      /* arg, we cannot handle NaN's. */
-      iv->min   = tarval_bad;
-      iv->max   = tarval_bad;
-      iv->flags = MIN_EXCLUDED | MAX_EXCLUDED;
-
-      return NULL;
-    }
-  }
-
-  /* check which side is known */
-  switch (pnc) {
-  case pn_Cmp_Eq:
-    /* [tv, tv] */
-    iv->min   =
-    iv->max   = tv;
-    iv->flags = MIN_INCLUDED | MAX_INCLUDED;
-    break;
-
-  case pn_Cmp_Le:
-    /* [-oo, tv] */
-    iv->min   = get_mode_min(mode);
-    iv->max   = tv;
-    iv->flags = MIN_INCLUDED | MAX_INCLUDED;
-    break;
-
-  case pn_Cmp_Lt:
-    /* [-oo, tv) */
-    iv->min   = get_mode_min(mode);
-    iv->max   = tv;
-    iv->flags = MIN_INCLUDED | MAX_EXCLUDED;
-    break;
-
-  case pn_Cmp_Gt:
-    /* (tv, +oo] */
-    iv->min   = tv;
-    iv->max   = get_mode_max(mode);
-    iv->flags = MIN_EXCLUDED | MAX_INCLUDED;
-    break;
-
-  case pn_Cmp_Ge:
-    /* [tv, +oo] */
-    iv->min   = tv;
-    iv->max   = get_mode_max(mode);
-    iv->flags = MIN_INCLUDED | MAX_INCLUDED;
-    break;
-
-  case pn_Cmp_Leg:
-    /*
-     * Ordered means, that at least neither
-     * our bound nor our value ara NaN's
-     */
-    /* [-oo, +oo] */
-    iv->min   = get_mode_min(mode);
-    iv->max   = get_mode_max(mode);
-    iv->flags = MIN_INCLUDED | MAX_INCLUDED;
-    break;
-
-  default:
-    /*
-     * We do not handle UNORDERED, as a NaN
-     * could be included in the interval.
-     */
-    iv->min   = tarval_bad;
-    iv->max   = tarval_bad;
-    iv->flags = MIN_EXCLUDED | MAX_EXCLUDED;
-    return NULL;
-  }
-
-  if (iv->min != tarval_bad && iv->max != tarval_bad)
-    return iv;
-  return NULL;
+static interval_t *get_interval(interval_t *iv, ir_node *bound, pn_Cmp pnc) {
+       ir_mode *mode = get_irn_mode(bound);
+       tarval  *tv   = value_of(bound);
+
+       if (tv == tarval_bad) {
+               /* There is nothing we could do here. For integer
+                * modes we could return [-oo, +oo], but there is
+                * nothing we could deduct from such an interval.
+                * So, speed things up and return unknown.
+                */
+               iv->min   = tarval_bad;
+               iv->max   = tarval_bad;
+               iv->flags = MIN_EXCLUDED | MAX_EXCLUDED;
+               return NULL;
+       }
+
+       if (mode_is_float(mode)) {
+               if (tv == get_mode_NAN(mode)) {
+                       /* arg, we cannot handle NaN's. */
+                       iv->min   = tarval_bad;
+                       iv->max   = tarval_bad;
+                       iv->flags = MIN_EXCLUDED | MAX_EXCLUDED;
+
+                       return NULL;
+               }
+       }
+
+       /* check which side is known */
+       switch (pnc) {
+       case pn_Cmp_Eq:
+               /* [tv, tv] */
+               iv->min   =
+                       iv->max   = tv;
+               iv->flags = MIN_INCLUDED | MAX_INCLUDED;
+               break;
+
+       case pn_Cmp_Le:
+               /* [-oo, tv] */
+               iv->min   = get_mode_min(mode);
+               iv->max   = tv;
+               iv->flags = MIN_INCLUDED | MAX_INCLUDED;
+               break;
+
+       case pn_Cmp_Lt:
+               /* [-oo, tv) */
+               iv->min   = get_mode_min(mode);
+               iv->max   = tv;
+               iv->flags = MIN_INCLUDED | MAX_EXCLUDED;
+               break;
+
+       case pn_Cmp_Gt:
+               /* (tv, +oo] */
+               iv->min   = tv;
+               iv->max   = get_mode_max(mode);
+               iv->flags = MIN_EXCLUDED | MAX_INCLUDED;
+               break;
+
+       case pn_Cmp_Ge:
+               /* [tv, +oo] */
+               iv->min   = tv;
+               iv->max   = get_mode_max(mode);
+               iv->flags = MIN_INCLUDED | MAX_INCLUDED;
+               break;
+
+       case pn_Cmp_Leg:
+               /*
+                * Ordered means, that at least neither
+                * our bound nor our value ara NaN's
+                */
+               /* [-oo, +oo] */
+               iv->min   = get_mode_min(mode);
+               iv->max   = get_mode_max(mode);
+               iv->flags = MIN_INCLUDED | MAX_INCLUDED;
+               break;
+
+       default:
+               /*
+                * We do not handle UNORDERED, as a NaN
+                * could be included in the interval.
+                */
+               iv->min   = tarval_bad;
+               iv->max   = tarval_bad;
+               iv->flags = MIN_EXCLUDED | MAX_EXCLUDED;
+               return NULL;
+       }
+
+       if (iv->min != tarval_bad && iv->max != tarval_bad)
+               return iv;
+       return NULL;
 }  /* get_interval */
 
 /**
@@ -437,136 +474,134 @@ static interval_t *get_interval(interval_t *iv, ir_node *bound, pn_Cmp pnc)
  *   tarval_b_true or tarval_b_false it it can be evaluated,
  *   tarval_bad else
  */
-static tarval *(compare_iv)(const interval_t *l_iv, const interval_t *r_iv, pn_Cmp pnc)
-{
-  pn_Cmp res;
-  unsigned flags;
-  tarval *tv_true = tarval_b_true, *tv_false = tarval_b_false;
-
-  /* if one interval contains NaNs, we cannot evaluate anything */
-  if (! l_iv || ! r_iv)
-    return tarval_bad;
-
-  /* we can only check ordered relations */
-  if (pnc & pn_Cmp_Uo) {
-    tarval *t;
-
-    pnc      = get_negated_pnc(pnc, get_tarval_mode(l_iv->min));
-    t        = tv_true;
-    tv_true  = tv_false;
-    tv_false = t;
-  }
-
-  /* if we have > or >=, we do the inverse to save some cases */
-  if (pnc == pn_Cmp_Ge || pnc == pn_Cmp_Gt) {
-    const interval_t *t;
-
-    pnc  = get_inversed_pnc(pnc);
-    t    = l_iv;
-    l_iv = r_iv;
-    r_iv = t;
-  }
-
-  /* now, only the following cases remains */
-  switch (pnc) {
-  case pn_Cmp_Eq:
-    /* two intervals can be compared for equality only if they are a single value */
-    if (l_iv->min == l_iv->max && r_iv->min == r_iv->max)
-      return tarval_cmp(l_iv->min, r_iv->min) == pn_Cmp_Eq ? tv_true : tv_false;
-
-    /* if both intervals do not intersect, it is never equal */
-    res = tarval_cmp(l_iv->max, r_iv->min);
-
-    /* b < c ==> [a,b] != [c,d] */
-    if (res == pn_Cmp_Lt)
-      return tv_false;
-
-    /* b <= c ==> [a,b) != [c,d]  AND [a,b] != (c,d] */
-    if ((l_iv->flags & MAX_EXCLUDED || r_iv->flags & MIN_EXCLUDED)
-        && (res == pn_Cmp_Eq))
-      return tv_false;
-
-    res = tarval_cmp(r_iv->max, l_iv->min);
-
-    /* d < a ==> [c,d] != [a,b] */
-    if (res == pn_Cmp_Lt)
-      return tv_false;
-
-    /* d <= a ==> [c,d) != [a,b]  AND [c,d] != (a,b] */
-    if ((r_iv->flags & MAX_EXCLUDED || l_iv->flags & MIN_EXCLUDED)
-        && (res == pn_Cmp_Eq))
-      return tv_false;
-    break;
-
-  case pn_Cmp_Lg:
-    /* two intervals can be compared for not equality only if they are a single value */
-    if (l_iv->min == l_iv->max && r_iv->min == r_iv->max)
-      return tarval_cmp(l_iv->min, r_iv->min) != pn_Cmp_Eq ? tv_true : tv_false;
-    break;
-
-  case pn_Cmp_Lt:
-    res = tarval_cmp(l_iv->max, r_iv->min);
-
-    /* [a, b] < [c, d]  <==> b < c */
-    if (res == pn_Cmp_Lt)
-      return tv_true;
-
-    /* if one border is excluded, b <= c is enough */
-    if ((l_iv->flags & MAX_EXCLUDED || r_iv->flags & MIN_EXCLUDED) &&
-        res == pn_Cmp_Eq)
-      return tv_true;
-
-    /* [a, b] >= [c, d] <==> a > d */
-    res = tarval_cmp(l_iv->min, r_iv->max);
-    if (res == pn_Cmp_Gt)
-      return tv_false;
-
-    /* if one border is excluded, a >= d is enough */
-    if ((l_iv->flags & MIN_EXCLUDED || r_iv->flags & MAX_EXCLUDED) &&
-      res == pn_Cmp_Eq)
-      return tv_false;
-    break;
-
-  case pn_Cmp_Le:
-    /* [a, b) <= [c, d] or [a, b] <= (c, d]  <==> b <= c */
-    flags = (l_iv->flags & MAX_EXCLUDED) | (r_iv->flags & MIN_EXCLUDED);
-    if (flags) {
-      res = tarval_cmp(l_iv->max, r_iv->min);
-
-      if (res == pn_Cmp_Lt || res == pn_Cmp_Eq)
-        return tv_true;
-    }
-
-    res = tarval_cmp(l_iv->min, r_iv->max);
-
-    /* [a, b] > [c, d] <==> a > d */
-    if (res == pn_Cmp_Gt)
-      return tv_false;
-
-    /* if one border is excluded, a >= d is enough */
-    if ((l_iv->flags & MIN_EXCLUDED || r_iv->flags & MAX_EXCLUDED) &&
-        res == pn_Cmp_Eq)
-      return tv_false;
-    break;
-
-  case pn_Cmp_Leg:
-    /* Hmm. if both are intervals, we can find an order */
-    return tv_true;
-
-  default:
-    return tarval_bad;
-  }
-  return tarval_bad;
+static tarval *(compare_iv)(const interval_t *l_iv, const interval_t *r_iv, pn_Cmp pnc) {
+       pn_Cmp res;
+       unsigned flags;
+       tarval *tv_true = tarval_b_true, *tv_false = tarval_b_false;
+
+       /* if one interval contains NaNs, we cannot evaluate anything */
+       if (! l_iv || ! r_iv)
+               return tarval_bad;
+
+       /* we can only check ordered relations */
+       if (pnc & pn_Cmp_Uo) {
+               tarval *t;
+
+               pnc      = get_negated_pnc(pnc, get_tarval_mode(l_iv->min));
+               t        = tv_true;
+               tv_true  = tv_false;
+               tv_false = t;
+       }
+
+       /* if we have > or >=, we do the inverse to save some cases */
+       if (pnc == pn_Cmp_Ge || pnc == pn_Cmp_Gt) {
+               const interval_t *t;
+
+               pnc  = get_inversed_pnc(pnc);
+               t    = l_iv;
+               l_iv = r_iv;
+               r_iv = t;
+       }
+
+       /* now, only the following cases remains */
+       switch (pnc) {
+       case pn_Cmp_Eq:
+               /* two intervals can be compared for equality only if they are a single value */
+               if (l_iv->min == l_iv->max && r_iv->min == r_iv->max)
+                       return tarval_cmp(l_iv->min, r_iv->min) == pn_Cmp_Eq ? tv_true : tv_false;
+
+               /* if both intervals do not intersect, it is never equal */
+               res = tarval_cmp(l_iv->max, r_iv->min);
+
+               /* b < c ==> [a,b] != [c,d] */
+               if (res == pn_Cmp_Lt)
+                       return tv_false;
+
+               /* b <= c ==> [a,b) != [c,d]  AND [a,b] != (c,d] */
+               if ((l_iv->flags & MAX_EXCLUDED || r_iv->flags & MIN_EXCLUDED)
+                       && (res == pn_Cmp_Eq))
+                       return tv_false;
+
+               res = tarval_cmp(r_iv->max, l_iv->min);
+
+               /* d < a ==> [c,d] != [a,b] */
+               if (res == pn_Cmp_Lt)
+                       return tv_false;
+
+               /* d <= a ==> [c,d) != [a,b]  AND [c,d] != (a,b] */
+               if ((r_iv->flags & MAX_EXCLUDED || l_iv->flags & MIN_EXCLUDED)
+                       && (res == pn_Cmp_Eq))
+                       return tv_false;
+               break;
+
+       case pn_Cmp_Lg:
+               /* two intervals can be compared for not equality only if they are a single value */
+               if (l_iv->min == l_iv->max && r_iv->min == r_iv->max)
+                       return tarval_cmp(l_iv->min, r_iv->min) != pn_Cmp_Eq ? tv_true : tv_false;
+               break;
+
+       case pn_Cmp_Lt:
+               res = tarval_cmp(l_iv->max, r_iv->min);
+
+               /* [a, b] < [c, d]  <==> b < c */
+               if (res == pn_Cmp_Lt)
+                       return tv_true;
+
+               /* if one border is excluded, b <= c is enough */
+               if ((l_iv->flags & MAX_EXCLUDED || r_iv->flags & MIN_EXCLUDED) &&
+                       res == pn_Cmp_Eq)
+                       return tv_true;
+
+               /* [a, b] >= [c, d] <==> a > d */
+               res = tarval_cmp(l_iv->min, r_iv->max);
+               if (res == pn_Cmp_Gt)
+                       return tv_false;
+
+               /* if one border is excluded, a >= d is enough */
+               if ((l_iv->flags & MIN_EXCLUDED || r_iv->flags & MAX_EXCLUDED) &&
+                       res == pn_Cmp_Eq)
+                       return tv_false;
+               break;
+
+       case pn_Cmp_Le:
+               /* [a, b) <= [c, d] or [a, b] <= (c, d]  <==> b <= c */
+               flags = (l_iv->flags & MAX_EXCLUDED) | (r_iv->flags & MIN_EXCLUDED);
+               if (flags) {
+                       res = tarval_cmp(l_iv->max, r_iv->min);
+
+                       if (res == pn_Cmp_Lt || res == pn_Cmp_Eq)
+                               return tv_true;
+               }
+
+               res = tarval_cmp(l_iv->min, r_iv->max);
+
+               /* [a, b] > [c, d] <==> a > d */
+               if (res == pn_Cmp_Gt)
+                       return tv_false;
+
+               /* if one border is excluded, a >= d is enough */
+               if ((l_iv->flags & MIN_EXCLUDED || r_iv->flags & MAX_EXCLUDED) &&
+                       res == pn_Cmp_Eq)
+                       return tv_false;
+               break;
+
+       case pn_Cmp_Leg:
+               /* Hmm. if both are intervals, we can find an order */
+               return tv_true;
+
+       default:
+               return tarval_bad;
+       }
+       return tarval_bad;
 }  /* compare_iv */
 
 /**
  * Returns non-zero, if a given relation is transitive.
  */
 static int is_transitive(pn_Cmp pnc) {
-  return (pn_Cmp_False < pnc && pnc < pn_Cmp_Lg);
+       return (pn_Cmp_False < pnc && pnc < pn_Cmp_Lg);
 }  /* is_transitive */
 
-
 /**
  * Return the value of a Cmp if one or both predecessors
  * are Confirm nodes.
@@ -576,167 +611,180 @@ static int is_transitive(pn_Cmp pnc) {
  * @param right  the right operand of the Cmp
  * @param pnc    the compare relation
  */
-tarval *computed_value_Cmp_Confirm(ir_node *cmp, ir_node *left, ir_node *right, pn_Cmp pnc)
-{
-  ir_node    *l_bound;
-  pn_Cmp     l_pnc, res_pnc, neg_pnc;
-  interval_t l_iv, r_iv;
-  tarval     *tv;
-  ir_mode    *mode;
-
-  if (get_irn_op(right) == op_Confirm) {
-    ir_node *t;
-
-    /* we want the Confirm on the left side */
-    t     = left;
-    left  = right;
-    right = t;
-
-    pnc = get_inversed_pnc(pnc);
-  }
-  else if (get_irn_op(left) != op_Confirm) {
-    /* no Confirm on either one side, finish */
-    return tarval_bad;
-  }
-
-  /* ok, here at least left is a Confirm, right might be */
-  l_bound = get_Confirm_bound(left);
-  l_pnc   = get_Confirm_cmp(left);
-
-  if (get_irn_op(right) == op_Confirm) {
-    /*
-     * both sides are Confirm's. Check some rare cases first.
-     */
-    ir_node *r_bound = get_Confirm_bound(right);
-    pn_Cmp  r_pnc = get_Confirm_cmp(right);
-
-    /*
-     * some check can be made WITHOUT constant bounds
-     */
-    if (r_bound == l_bound) {
-      if (is_transitive(l_pnc)) {
-        pn_Cmp r_inc_pnc = get_inversed_pnc(r_pnc);
-
-        /*
-         * triangle inequality:
-         *
-         * a CMP B && B CMP b => a CMP b, !(a ~CMP b)
-         *
-         * We handle correctly cases with some <=/>= here
-         */
-        if ((l_pnc & ~pn_Cmp_Eq) == (r_inc_pnc & ~pn_Cmp_Eq)) {
-          res_pnc = (l_pnc & ~pn_Cmp_Eq) | (l_pnc & r_inc_pnc & pn_Cmp_Eq);
-
-          if ((pnc == res_pnc) || ((pnc & ~pn_Cmp_Eq) == res_pnc)) {
-            DBG_OUT_TR(l_pnc, l_bound, r_pnc, r_bound, pnc, "true");
-            DBG_EVAL_CONFIRM(cmp);
-            return tarval_b_true;
-          }
-          else {
-            pn_Cmp neg_pnc = get_negated_pnc(pnc, get_irn_mode(left));
-
-            if ((neg_pnc == res_pnc) || ((neg_pnc & ~pn_Cmp_Eq) == res_pnc)) {
-              DBG_OUT_TR(l_pnc, l_bound, r_pnc, r_bound, pnc, "false");
-              DBG_EVAL_CONFIRM(cmp);
-              return tarval_b_false;
-            }
-          }
-        }
-      }
-    }
-
-    /*
-     * Here, we check only the right Confirm, as the left Confirms are
-     * checked later anyway.
-     */
-
-    if (left == r_bound) {
-      /*
-       * l == bound(r) AND pnc(r) == pnc:
-       *
-       * We know that a CMP b and check for that
-       */
-      if ((r_pnc == pnc) || (r_pnc == (pnc & ~pn_Cmp_Eq))) {
-        DBG_OUT_R(r_pnc, r_bound, left, pnc, right, "true");
-        DBG_EVAL_CONFIRM(cmp);
-        return tarval_b_true;
-      }
-      /*
-       * l == bound(r) AND pnc(r) != pnc:
-       *
-       * We know that a CMP b and check for a ~CMP b
-       */
-      else {
-        mode    = get_irn_mode(left);
-        neg_pnc = get_negated_pnc(pnc, mode);
-
-        if ((r_pnc == neg_pnc) || (r_pnc == (neg_pnc & ~pn_Cmp_Eq))) {
-          DBG_OUT_R(r_pnc, r_bound, left, pnc, right, "false");
-          DBG_EVAL_CONFIRM(cmp);
-          return tarval_b_false;
-        }
-      }
-    }
-
-    /* now, try interval magic */
-    tv = compare_iv(
-      get_interval(&l_iv, l_bound, l_pnc),
-      get_interval(&r_iv, r_bound, r_pnc),
-      pnc);
-
-    if (tv != tarval_bad) {
-      DBG_EVAL_CONFIRM(cmp);
-      return tv;
-    }
-  }
-
-  /* from Here, check only left Confirm */
-
-  /*
-   * some checks can be made WITHOUT constant bounds
-   */
-  if (right == l_bound) {
-    /*
-     * r == bound(l) AND pnc(l) == pnc:
-     *
-     * We know that a CMP b and check for that
-     */
-    if ((l_pnc == pnc) || (l_pnc == (pnc & ~pn_Cmp_Eq))) {
-      DBG_OUT_L(l_pnc, l_bound, left, pnc, right, "true");
-      DBG_EVAL_CONFIRM(cmp);
-      return tarval_b_true;
-    }
-    /*
-     * r == bound(l) AND pnc(l) is Not(pnc):
-     *
-     * We know that a CMP b and check for a ~CMP b
-     */
-    else {
-      mode = get_irn_mode(left);
-      neg_pnc = get_negated_pnc(pnc, mode);
-
-      if ((l_pnc == neg_pnc) || (l_pnc == (neg_pnc & ~pn_Cmp_Eq))) {
-        DBG_OUT_L(l_pnc, l_bound, left, pnc, right, "false");
-        DBG_EVAL_CONFIRM(cmp);
-        return tarval_b_false;
-      }
-    }
-  }
-
-  /* now, only right == Const can help */
-  tv = value_of(right);
-
-  if (tv != tarval_bad) {
-    tv = compare_iv(
-      get_interval(&l_iv, l_bound, l_pnc),
-      get_interval_from_tv(&r_iv, tv),
-      pnc);
-  }
-
-  if (tv != tarval_bad)
-    DBG_EVAL_CONFIRM(cmp);
-
-  return tv;
+tarval *computed_value_Cmp_Confirm(ir_node *cmp, ir_node *left, ir_node *right, pn_Cmp pnc) {
+       ir_node         *l_bound;
+       pn_Cmp          l_pnc, res_pnc, neg_pnc;
+       interval_t      l_iv, r_iv;
+       tarval          *tv;
+       ir_mode         *mode;
+
+       if (is_Confirm(right)) {
+               /* we want the Confirm on the left side */
+               ir_node *t = right;
+               right = left;
+               left  = t;
+
+               pnc = get_inversed_pnc(pnc);
+       } else if (! is_Confirm(left)) {
+               /* nothing more found */
+               tv = tarval_bad;
+               goto check_null_case;
+       }
+
+       /* ok, here at least left is a Confirm, right might be */
+       l_bound = get_Confirm_bound(left);
+       l_pnc   = get_Confirm_cmp(left);
+
+       if (is_Confirm(right)) {
+               /*
+                * both sides are Confirm's. Check some rare cases first.
+                */
+               ir_node *r_bound = get_Confirm_bound(right);
+               pn_Cmp  r_pnc    = get_Confirm_cmp(right);
+
+               /*
+                * some check can be made WITHOUT constant bounds
+                */
+               if (r_bound == l_bound) {
+                       if (is_transitive(l_pnc)) {
+                               pn_Cmp r_inc_pnc = get_inversed_pnc(r_pnc);
+
+                               /*
+                                * triangle inequality:
+                                *
+                                * a CMP B && B CMP b => a CMP b, !(a ~CMP b)
+                                *
+                                * We handle correctly cases with some <=/>= here
+                                */
+                               if ((l_pnc & ~pn_Cmp_Eq) == (r_inc_pnc & ~pn_Cmp_Eq)) {
+                                       res_pnc = (l_pnc & ~pn_Cmp_Eq) | (l_pnc & r_inc_pnc & pn_Cmp_Eq);
+
+                                       if ((pnc == res_pnc) || ((pnc & ~pn_Cmp_Eq) == res_pnc)) {
+                                               DBG_OUT_TR(l_pnc, l_bound, r_pnc, r_bound, pnc, "true");
+                                               DBG_EVAL_CONFIRM(cmp);
+                                               return tarval_b_true;
+                                       } else {
+                                               pn_Cmp neg_pnc = get_negated_pnc(pnc, get_irn_mode(left));
+
+                                               if ((neg_pnc == res_pnc) || ((neg_pnc & ~pn_Cmp_Eq) == res_pnc)) {
+                                                       DBG_OUT_TR(l_pnc, l_bound, r_pnc, r_bound, pnc, "false");
+                                                       DBG_EVAL_CONFIRM(cmp);
+                                                       return tarval_b_false;
+                                               }
+                                       }
+                               }
+                       }
+               }
+
+               /*
+                * Here, we check only the right Confirm, as the left Confirms are
+                * checked later anyway.
+                */
+               if (left == r_bound) {
+                       /*
+                        * l == bound(r) AND pnc(r) == pnc:
+                        *
+                        * We know that a CMP b and check for that
+                        */
+                       if ((r_pnc == pnc) || (r_pnc == (pnc & ~pn_Cmp_Eq))) {
+                               DBG_OUT_R(r_pnc, r_bound, left, pnc, right, "true");
+                               DBG_EVAL_CONFIRM(cmp);
+                               return tarval_b_true;
+                       }
+                       /*
+                        * l == bound(r) AND pnc(r) != pnc:
+                        *
+                        * We know that a CMP b and check for a ~CMP b
+                        */
+                       else {
+                               mode    = get_irn_mode(left);
+                               neg_pnc = get_negated_pnc(pnc, mode);
+
+                               if ((r_pnc == neg_pnc) || (r_pnc == (neg_pnc & ~pn_Cmp_Eq))) {
+                                       DBG_OUT_R(r_pnc, r_bound, left, pnc, right, "false");
+                                       DBG_EVAL_CONFIRM(cmp);
+                                       return tarval_b_false;
+                               }
+                       }
+               }
+
+               /* now, try interval magic */
+               tv = compare_iv(
+                       get_interval(&l_iv, l_bound, l_pnc),
+                       get_interval(&r_iv, r_bound, r_pnc),
+                       pnc);
+
+               if (tv != tarval_bad) {
+                       DBG_EVAL_CONFIRM(cmp);
+                       return tv;
+               }
+       }
+
+       /* from Here, check only left Confirm */
+
+       /*
+        * some checks can be made WITHOUT constant bounds
+        */
+       if (right == l_bound) {
+               /*
+                * r == bound(l) AND pnc(l) == pnc:
+                *
+                * We know that a CMP b and check for that
+                */
+               if ((l_pnc == pnc) || (l_pnc == (pnc & ~pn_Cmp_Eq))) {
+                       DBG_OUT_L(l_pnc, l_bound, left, pnc, right, "true");
+                       DBG_EVAL_CONFIRM(cmp);
+                       return tarval_b_true;
+               }
+               /*
+                * r == bound(l) AND pnc(l) is Not(pnc):
+                *
+                * We know that a CMP b and check for a ~CMP b
+                */
+               else {
+                       mode = get_irn_mode(left);
+                       neg_pnc = get_negated_pnc(pnc, mode);
+
+                       if ((l_pnc == neg_pnc) || (l_pnc == (neg_pnc & ~pn_Cmp_Eq))) {
+                               DBG_OUT_L(l_pnc, l_bound, left, pnc, right, "false");
+                               DBG_EVAL_CONFIRM(cmp);
+                               return tarval_b_false;
+                       }
+               }
+       }
+
+       /* now, only right == Const can help */
+       tv = value_of(right);
+
+       if (tv != tarval_bad) {
+               tv = compare_iv(
+                       get_interval(&l_iv, l_bound, l_pnc),
+                       get_interval_from_tv(&r_iv, tv),
+                       pnc);
+       } else {
+check_null_case:
+               /* check some other cases */
+               if ((pnc == pn_Cmp_Eq || pnc == pn_Cmp_Lg) &&
+                       is_Const(right) && is_Const_null(right)) {
+                       /* for == 0 or != 0 we have some special tools */
+                       ir_mode       *mode = get_irn_mode(left);
+                       const ir_node *dummy;
+                       if (mode_is_reference(mode)) {
+                               if (value_not_null(left, &dummy)) {
+                                       tv = pnc == pn_Cmp_Eq ? tarval_b_false : tarval_b_true;
+                               }
+                       } else {
+                               if (value_not_zero(left, &dummy)) {
+                                       tv = pnc == pn_Cmp_Eq ? tarval_b_false : tarval_b_true;
+                               }
+                       }
+               }
+       }
+
+       if (tv != tarval_bad)
+               DBG_EVAL_CONFIRM(cmp);
+
+       return tv;
 }  /* computed_value_Cmp_Confirm */
 
 #ifdef DEBUG_CONFIRM
@@ -748,24 +796,23 @@ tarval *computed_value_Cmp_Confirm(ir_node *cmp, ir_node *left, ir_node *right,
  * @param iv    the interval
  */
 static int iv_snprintf(char *buf, size_t len, const interval_t *iv) {
-  char smin[64], smax[64];
-
-  if (iv) {
-    tarval_snprintf(smin, sizeof(smin), iv->min);
-
-    if (iv->min != iv->max || (iv->flags & (MIN_EXCLUDED|MAX_EXCLUDED))) {
-      tarval_snprintf(smax, sizeof(smax), iv->max);
-
-      return snprintf(buf, len, "%c%s, %s%c",
-              iv->flags & MIN_EXCLUDED ? '(' : '[',
-              smin, smax,
-              iv->flags & MAX_EXCLUDED ? ')' : ']'
-      );
-    }
-    else
-      return snprintf(buf, len, "%s", smin);
-  }
-  return snprintf(buf, len, "<UNKNOWN>");
+       char smin[64], smax[64];
+
+       if (iv) {
+               tarval_snprintf(smin, sizeof(smin), iv->min);
+
+               if (iv->min != iv->max || (iv->flags & (MIN_EXCLUDED|MAX_EXCLUDED))) {
+                       tarval_snprintf(smax, sizeof(smax), iv->max);
+
+                       return snprintf(buf, len, "%c%s, %s%c",
+                               iv->flags & MIN_EXCLUDED ? '(' : '[',
+                               smin, smax,
+                               iv->flags & MAX_EXCLUDED ? ')' : ']'
+                               );
+               } else
+                       return snprintf(buf, len, "%s", smin);
+       }
+       return snprintf(buf, len, "<UNKNOWN>");
 }  /* iv_snprintf */
 
 /**
@@ -775,14 +822,13 @@ static int iv_snprintf(char *buf, size_t len, const interval_t *iv) {
  * @param r_iv  the right interval
  * @param pnc   the compare relation
  */
-static void print_iv_cmp(const interval_t *l_iv, const interval_t *r_iv, pn_Cmp pnc)
-{
-  char sl[128], sr[128];
+static void print_iv_cmp(const interval_t *l_iv, const interval_t *r_iv, pn_Cmp pnc) {
+       char sl[128], sr[128];
 
-  iv_snprintf(sl, sizeof(sl), l_iv);
-  iv_snprintf(sr, sizeof(sr), r_iv);
+       iv_snprintf(sl, sizeof(sl), l_iv);
+       iv_snprintf(sr, sizeof(sr), r_iv);
 
-  ir_printf("%s %= %s", sl, pnc, sr);
+       ir_printf("%s %= %s", sl, pnc, sr);
 }  /* print_iv_cmp */
 
 /**
@@ -792,17 +838,16 @@ static void print_iv_cmp(const interval_t *l_iv, const interval_t *r_iv, pn_Cmp
  * @param r_iv  the right interval
  * @param pnc   the compare relation
  */
-static tarval *compare_iv_dbg(const interval_t *l_iv, const interval_t *r_iv, pn_Cmp pnc)
-{
-  tarval *tv = (compare_iv)(l_iv, r_iv, pnc);
+static tarval *compare_iv_dbg(const interval_t *l_iv, const interval_t *r_iv, pn_Cmp pnc) {
+       tarval *tv = (compare_iv)(l_iv, r_iv, pnc);
 
-  if (tv == tarval_bad)
-    return tv;
+       if (tv == tarval_bad)
+       return tv;
 
-  ir_printf("In %e:\n", get_irg_entity(current_ir_graph));
-  print_iv_cmp(l_iv, r_iv, pnc);
-  ir_printf(" = %T\n", tv);
-  return tv;
+       ir_printf("In %e:\n", get_irg_entity(current_ir_graph));
+       print_iv_cmp(l_iv, r_iv, pnc);
+       ir_printf(" = %T\n", tv);
+       return tv;
 }  /* compare_iv_dbg */
 
 #endif /* DEBUG_CONFIRM */