isl_pw_templ.c: rename align_params_pw_pw_and to include PW
[platform/upstream/isl.git] / isl_aff.c
index 8c81c08..3379d01 100644 (file)
--- a/isl_aff.c
+++ b/isl_aff.c
@@ -12,7 +12,7 @@
 #include <isl_ctx_private.h>
 #include <isl_map_private.h>
 #include <isl_aff_private.h>
-#include <isl_dim_private.h>
+#include <isl_space_private.h>
 #include <isl_local_space_private.h>
 #include <isl_mat_private.h>
 #include <isl_list_private.h>
@@ -56,6 +56,10 @@ __isl_give isl_aff *isl_aff_alloc(__isl_take isl_local_space *ls)
        if (!isl_local_space_divs_known(ls))
                isl_die(ctx, isl_error_invalid, "local space has unknown divs",
                        goto error);
+       if (!isl_local_space_is_set(ls))
+               isl_die(ctx, isl_error_invalid,
+                       "domain of affine expression should be a set",
+                       goto error);
 
        total = isl_local_space_dim(ls, isl_dim_all);
        v = isl_vec_alloc(ctx, 1 + 1 + total);
@@ -65,7 +69,7 @@ error:
        return NULL;
 }
 
-__isl_give isl_aff *isl_aff_zero(__isl_take isl_local_space *ls)
+__isl_give isl_aff *isl_aff_zero_on_domain(__isl_take isl_local_space *ls)
 {
        isl_aff *aff;
 
@@ -129,45 +133,97 @@ isl_ctx *isl_aff_get_ctx(__isl_keep isl_aff *aff)
        return aff ? isl_local_space_get_ctx(aff->ls) : NULL;
 }
 
+/* Externally, an isl_aff has a map space, but internally, the
+ * ls field corresponds to the domain of that space.
+ */
 int isl_aff_dim(__isl_keep isl_aff *aff, enum isl_dim_type type)
 {
-       return aff ? isl_local_space_dim(aff->ls, type) : 0;
+       if (!aff)
+               return 0;
+       if (type == isl_dim_out)
+               return 1;
+       if (type == isl_dim_in)
+               type = isl_dim_set;
+       return isl_local_space_dim(aff->ls, type);
 }
 
-__isl_give isl_dim *isl_aff_get_dim(__isl_keep isl_aff *aff)
+__isl_give isl_space *isl_aff_get_domain_space(__isl_keep isl_aff *aff)
 {
-       return aff ? isl_local_space_get_dim(aff->ls) : NULL;
+       return aff ? isl_local_space_get_space(aff->ls) : NULL;
 }
 
-__isl_give isl_local_space *isl_aff_get_local_space(__isl_keep isl_aff *aff)
+__isl_give isl_space *isl_aff_get_space(__isl_keep isl_aff *aff)
+{
+       isl_space *space;
+       if (!aff)
+               return NULL;
+       space = isl_local_space_get_space(aff->ls);
+       space = isl_space_from_domain(space);
+       space = isl_space_add_dims(space, isl_dim_out, 1);
+       return space;
+}
+
+__isl_give isl_local_space *isl_aff_get_domain_local_space(
+       __isl_keep isl_aff *aff)
 {
        return aff ? isl_local_space_copy(aff->ls) : NULL;
 }
 
+__isl_give isl_local_space *isl_aff_get_local_space(__isl_keep isl_aff *aff)
+{
+       isl_local_space *ls;
+       if (!aff)
+               return NULL;
+       ls = isl_local_space_copy(aff->ls);
+       ls = isl_local_space_from_domain(ls);
+       ls = isl_local_space_add_dims(ls, isl_dim_out, 1);
+       return ls;
+}
+
+/* Externally, an isl_aff has a map space, but internally, the
+ * ls field corresponds to the domain of that space.
+ */
 const char *isl_aff_get_dim_name(__isl_keep isl_aff *aff,
        enum isl_dim_type type, unsigned pos)
 {
-       return aff ? isl_local_space_get_dim_name(aff->ls, type, pos) : 0;
+       if (!aff)
+               return NULL;
+       if (type == isl_dim_out)
+               return NULL;
+       if (type == isl_dim_in)
+               type = isl_dim_set;
+       return isl_local_space_get_dim_name(aff->ls, type, pos);
 }
 
-__isl_give isl_aff *isl_aff_reset_dim(__isl_take isl_aff *aff,
-       __isl_take isl_dim *dim)
+__isl_give isl_aff *isl_aff_reset_domain_space(__isl_take isl_aff *aff,
+       __isl_take isl_space *dim)
 {
        aff = isl_aff_cow(aff);
        if (!aff || !dim)
                goto error;
 
-       aff->ls = isl_local_space_reset_dim(aff->ls, dim);
+       aff->ls = isl_local_space_reset_space(aff->ls, dim);
        if (!aff->ls)
                return isl_aff_free(aff);
 
        return aff;
 error:
        isl_aff_free(aff);
-       isl_dim_free(dim);
+       isl_space_free(dim);
        return NULL;
 }
 
+/* Reset the space of "aff".  This function is called from isl_pw_templ.c
+ * and doesn't know if the space of an element object is represented
+ * directly or through its domain.  It therefore passes along both.
+ */
+__isl_give isl_aff *isl_aff_reset_space_and_domain(__isl_take isl_aff *aff,
+       __isl_take isl_space *space, __isl_take isl_space *domain)
+{
+       isl_space_free(space);
+       return isl_aff_reset_domain_space(aff, domain);
+}
+
 /* Reorder the coefficients of the affine expression based
  * on the given reodering.
  * The reordering r is assumed to have been extended with the local
@@ -182,7 +238,8 @@ static __isl_give isl_vec *vec_reorder(__isl_take isl_vec *vec,
        if (!vec || !r)
                goto error;
 
-       res = isl_vec_alloc(vec->ctx, 2 + isl_dim_total(r->dim) + n_div);
+       res = isl_vec_alloc(vec->ctx,
+                           2 + isl_space_dim(r->dim, isl_dim_all) + n_div);
        isl_seq_cpy(res->el, vec->el, 2);
        isl_seq_clr(res->el + 2, res->size - 2);
        for (i = 0; i < r->len; ++i)
@@ -197,9 +254,10 @@ error:
        return NULL;
 }
 
-/* Reorder the dimensions of "aff" according to the given reordering.
+/* Reorder the dimensions of the domain of "aff" according
+ * to the given reordering.
  */
-__isl_give isl_aff *isl_aff_realign(__isl_take isl_aff *aff,
+__isl_give isl_aff *isl_aff_realign_domain(__isl_take isl_aff *aff,
        __isl_take isl_reordering *r)
 {
        aff = isl_aff_cow(aff);
@@ -265,6 +323,13 @@ int isl_aff_get_coefficient(__isl_keep isl_aff *aff,
        if (!aff)
                return -1;
 
+       if (type == isl_dim_out)
+               isl_die(aff->v->ctx, isl_error_invalid,
+                       "output/set dimension does not have a coefficient",
+                       return -1);
+       if (type == isl_dim_in)
+               type = isl_dim_set;
+
        if (pos >= isl_local_space_dim(aff->ls, type))
                isl_die(aff->v->ctx, isl_error_invalid,
                        "position out of bounds", return -1);
@@ -356,6 +421,13 @@ __isl_give isl_aff *isl_aff_set_coefficient(__isl_take isl_aff *aff,
        if (!aff)
                return NULL;
 
+       if (type == isl_dim_out)
+               isl_die(aff->v->ctx, isl_error_invalid,
+                       "output/set dimension does not have a coefficient",
+                       return isl_aff_free(aff));
+       if (type == isl_dim_in)
+               type = isl_dim_set;
+
        if (pos >= isl_local_space_dim(aff->ls, type))
                isl_die(aff->v->ctx, isl_error_invalid,
                        "position out of bounds", return isl_aff_free(aff));
@@ -380,6 +452,13 @@ __isl_give isl_aff *isl_aff_set_coefficient_si(__isl_take isl_aff *aff,
        if (!aff)
                return NULL;
 
+       if (type == isl_dim_out)
+               isl_die(aff->v->ctx, isl_error_invalid,
+                       "output/set dimension does not have a coefficient",
+                       return isl_aff_free(aff));
+       if (type == isl_dim_in)
+               type = isl_dim_set;
+
        if (pos >= isl_local_space_dim(aff->ls, type))
                isl_die(aff->v->ctx, isl_error_invalid,
                        "position out of bounds", return isl_aff_free(aff));
@@ -404,6 +483,13 @@ __isl_give isl_aff *isl_aff_add_coefficient(__isl_take isl_aff *aff,
        if (!aff)
                return NULL;
 
+       if (type == isl_dim_out)
+               isl_die(aff->v->ctx, isl_error_invalid,
+                       "output/set dimension does not have a coefficient",
+                       return isl_aff_free(aff));
+       if (type == isl_dim_in)
+               type = isl_dim_set;
+
        if (pos >= isl_local_space_dim(aff->ls, type))
                isl_die(aff->v->ctx, isl_error_invalid,
                        "position out of bounds", return isl_aff_free(aff));
@@ -435,7 +521,7 @@ __isl_give isl_aff *isl_aff_add_coefficient_si(__isl_take isl_aff *aff,
        return aff;
 }
 
-__isl_give isl_div *isl_aff_get_div(__isl_keep isl_aff *aff, int pos)
+__isl_give isl_aff *isl_aff_get_div(__isl_keep isl_aff *aff, int pos)
 {
        if (!aff)
                return NULL;
@@ -660,7 +746,7 @@ __isl_give isl_aff *isl_aff_add(__isl_take isl_aff *aff1,
                goto error;
 
        ctx = isl_aff_get_ctx(aff1);
-       if (!isl_dim_equal(aff1->ls->dim, aff2->ls->dim))
+       if (!isl_space_is_equal(aff1->ls->dim, aff2->ls->dim))
                isl_die(ctx, isl_error_invalid,
                        "spaces don't match", goto error);
 
@@ -763,6 +849,12 @@ __isl_give isl_aff *isl_aff_set_dim_name(__isl_take isl_aff *aff,
        aff = isl_aff_cow(aff);
        if (!aff)
                return NULL;
+       if (type == isl_dim_out)
+               isl_die(aff->v->ctx, isl_error_invalid,
+                       "cannot set name of output/set dimension",
+                       return isl_aff_free(aff));
+       if (type == isl_dim_in)
+               type = isl_dim_set;
        aff->ls = isl_local_space_set_dim_name(aff->ls, type, pos, s);
        if (!aff->ls)
                return isl_aff_free(aff);
@@ -770,6 +862,29 @@ __isl_give isl_aff *isl_aff_set_dim_name(__isl_take isl_aff *aff,
        return aff;
 }
 
+__isl_give isl_aff *isl_aff_set_dim_id(__isl_take isl_aff *aff,
+       enum isl_dim_type type, unsigned pos, __isl_take isl_id *id)
+{
+       aff = isl_aff_cow(aff);
+       if (!aff)
+               return isl_id_free(id);
+       if (type == isl_dim_out)
+               isl_die(aff->v->ctx, isl_error_invalid,
+                       "cannot set name of output/set dimension",
+                       goto error);
+       if (type == isl_dim_in)
+               type = isl_dim_set;
+       aff->ls = isl_local_space_set_dim_id(aff->ls, type, pos, id);
+       if (!aff->ls)
+               return isl_aff_free(aff);
+
+       return aff;
+error:
+       isl_id_free(id);
+       isl_aff_free(aff);
+       return NULL;
+}
+
 /* Exploit the equalities in "eq" to simplify the affine expression
  * and the expressions of the integer divisions in the local space.
  * The integer divisions in this local space are assumed to appear
@@ -798,7 +913,7 @@ static __isl_give isl_aff *isl_aff_substitute_equalities_lifted(
        if (!aff->ls)
                goto error;
 
-       total = 1 + isl_dim_total(eq->dim);
+       total = 1 + isl_space_dim(eq->dim, isl_dim_all);
        n_div = eq->n_div;
        for (i = 0; i < eq->n_eq; ++i) {
                j = isl_seq_last_non_zero(eq->eq[i], total + n_div);
@@ -853,9 +968,10 @@ __isl_give isl_aff *isl_aff_gist(__isl_take isl_aff *aff,
        n_div = isl_local_space_dim(aff->ls, isl_dim_div);
        if (n_div > 0) {
                isl_basic_set *bset;
+               isl_local_space *ls;
                context = isl_set_add_dims(context, isl_dim_set, n_div);
-               bset = isl_basic_set_from_local_space(
-                                           isl_aff_get_local_space(aff));
+               ls = isl_aff_get_domain_local_space(aff);
+               bset = isl_basic_set_from_local_space(ls);
                bset = isl_basic_set_lift(bset);
                bset = isl_basic_set_flatten(bset);
                context = isl_set_intersect(context,
@@ -905,6 +1021,15 @@ __isl_give isl_basic_set *isl_aff_ge_basic_set(__isl_take isl_aff *aff1,
        return isl_aff_nonneg_basic_set(aff1);
 }
 
+/* Return a basic set containing those elements in the shared space
+ * of aff1 and aff2 where aff1 is smaller than or equal to aff2.
+ */
+__isl_give isl_basic_set *isl_aff_le_basic_set(__isl_take isl_aff *aff1,
+       __isl_take isl_aff *aff2)
+{
+       return isl_aff_ge_basic_set(aff2, aff1);
+}
+
 __isl_give isl_aff *isl_aff_add_on_domain(__isl_keep isl_set *dom,
        __isl_take isl_aff *aff1, __isl_take isl_aff *aff2)
 {
@@ -921,34 +1046,6 @@ int isl_aff_is_empty(__isl_keep isl_aff *aff)
        return 0;
 }
 
-/* Set active[i] to 1 if the dimension at position i is involved
- * in the affine expression.
- */
-static int set_active(__isl_keep isl_aff *aff, int *active)
-{
-       int i, j;
-       unsigned total;
-       unsigned offset;
-
-       if (!aff || !active)
-               return -1;
-
-       total = aff->v->size - 2;
-       for (i = 0; i < total; ++i)
-               active[i] = !isl_int_is_zero(aff->v->el[2 + i]);
-
-       offset = isl_local_space_offset(aff->ls, isl_dim_div) - 1;
-       for (i = aff->ls->div->n_row - 1; i >= 0; --i) {
-               if (!active[offset + i])
-                       continue;
-               for (j = 0; j < total; ++j)
-                       active[j] |=
-                               !isl_int_is_zero(aff->ls->div->row[i][2 + j]);
-       }
-
-       return 0;
-}
-
 /* Check whether the given affine expression has non-zero coefficient
  * for any dimension in the given range or if any of these dimensions
  * appear with non-zero coefficients in any of the integer divisions
@@ -972,9 +1069,8 @@ int isl_aff_involves_dims(__isl_keep isl_aff *aff,
                isl_die(ctx, isl_error_invalid,
                        "range out of bounds", return -1);
 
-       active = isl_calloc_array(ctx, int,
-                                 isl_local_space_dim(aff->ls, isl_dim_all));
-       if (set_active(aff, active) < 0)
+       active = isl_local_space_get_active(aff->ls, aff->v->el + 2);
+       if (!active)
                goto error;
 
        first += isl_local_space_offset(aff->ls, type) - 1;
@@ -999,11 +1095,17 @@ __isl_give isl_aff *isl_aff_drop_dims(__isl_take isl_aff *aff,
 
        if (!aff)
                return NULL;
+       if (type == isl_dim_out)
+               isl_die(aff->v->ctx, isl_error_invalid,
+                       "cannot drop output/set dimension",
+                       return isl_aff_free(aff));
+       if (type == isl_dim_in)
+               type = isl_dim_set;
        if (n == 0 && !isl_local_space_is_named_or_nested(aff->ls, type))
                return aff;
 
        ctx = isl_aff_get_ctx(aff);
-       if (first + n > isl_aff_dim(aff, type))
+       if (first + n > isl_local_space_dim(aff->ls, type))
                isl_die(ctx, isl_error_invalid, "range out of bounds",
                        return isl_aff_free(aff));
 
@@ -1030,11 +1132,17 @@ __isl_give isl_aff *isl_aff_insert_dims(__isl_take isl_aff *aff,
 
        if (!aff)
                return NULL;
+       if (type == isl_dim_out)
+               isl_die(aff->v->ctx, isl_error_invalid,
+                       "cannot insert output/set dimensions",
+                       return isl_aff_free(aff));
+       if (type == isl_dim_in)
+               type = isl_dim_set;
        if (n == 0 && !isl_local_space_is_named_or_nested(aff->ls, type))
                return aff;
 
        ctx = isl_aff_get_ctx(aff);
-       if (first > isl_aff_dim(aff, type))
+       if (first > isl_local_space_dim(aff->ls, type))
                isl_die(ctx, isl_error_invalid, "position out of bounds",
                        return isl_aff_free(aff));
 
@@ -1074,20 +1182,9 @@ __isl_give isl_pw_aff *isl_pw_aff_add_dims(__isl_take isl_pw_aff *pwaff,
        return isl_pw_aff_insert_dims(pwaff, type, pos, n);
 }
 
-__isl_give isl_pw_aff *isl_pw_aff_set_tuple_id(__isl_take isl_pw_aff *pwaff,
-       __isl_take isl_id *id)
-{
-       isl_dim *dim;
-
-       dim = isl_pw_aff_get_dim(pwaff);
-       dim = isl_dim_set_tuple_id(dim, isl_dim_set, id);
-
-       return isl_pw_aff_reset_dim(pwaff, dim);
-}
-
 __isl_give isl_pw_aff *isl_pw_aff_from_aff(__isl_take isl_aff *aff)
 {
-       isl_set *dom = isl_set_universe(isl_aff_get_dim(aff));
+       isl_set *dom = isl_set_universe(isl_aff_get_domain_space(aff));
        return isl_pw_aff_alloc(dom, aff);
 }
 
@@ -1119,15 +1216,15 @@ static __isl_give isl_set *align_params_pw_pw_set_and(
 {
        if (!pwaff1 || !pwaff2)
                goto error;
-       if (isl_dim_match(pwaff1->dim, isl_dim_param,
+       if (isl_space_match(pwaff1->dim, isl_dim_param,
                          pwaff2->dim, isl_dim_param))
                return fn(pwaff1, pwaff2);
-       if (!isl_dim_has_named_params(pwaff1->dim) ||
-           !isl_dim_has_named_params(pwaff2->dim))
+       if (!isl_space_has_named_params(pwaff1->dim) ||
+           !isl_space_has_named_params(pwaff2->dim))
                isl_die(isl_pw_aff_get_ctx(pwaff1), isl_error_invalid,
                        "unaligned unnamed parameters", goto error);
-       pwaff1 = isl_pw_aff_align_params(pwaff1, isl_pw_aff_get_dim(pwaff2));
-       pwaff2 = isl_pw_aff_align_params(pwaff2, isl_pw_aff_get_dim(pwaff1));
+       pwaff1 = isl_pw_aff_align_params(pwaff1, isl_pw_aff_get_space(pwaff2));
+       pwaff2 = isl_pw_aff_align_params(pwaff2, isl_pw_aff_get_space(pwaff1));
        return fn(pwaff1, pwaff2);
 error:
        isl_pw_aff_free(pwaff1);
@@ -1137,12 +1234,15 @@ error:
 
 /* Compute a piecewise quasi-affine expression with a domain that
  * is the union of those of pwaff1 and pwaff2 and such that on each
- * cell, the quasi-affine expression is the maximum of those of pwaff1
- * and pwaff2.  If only one of pwaff1 or pwaff2 is defined on a given
- * cell, then the associated expression is the defined one.
+ * cell, the quasi-affine expression is the better (according to cmp)
+ * of those of pwaff1 and pwaff2.  If only one of pwaff1 or pwaff2
+ * is defined on a given cell, then the associated expression
+ * is the defined one.
  */
-static __isl_give isl_pw_aff *pw_aff_union_max(__isl_take isl_pw_aff *pwaff1,
-       __isl_take isl_pw_aff *pwaff2)
+static __isl_give isl_pw_aff *pw_aff_union_opt(__isl_take isl_pw_aff *pwaff1,
+       __isl_take isl_pw_aff *pwaff2,
+       __isl_give isl_basic_set *(*cmp)(__isl_take isl_aff *aff1,
+                                       __isl_take isl_aff *aff2))
 {
        int i, j, n;
        isl_pw_aff *res;
@@ -1152,8 +1252,8 @@ static __isl_give isl_pw_aff *pw_aff_union_max(__isl_take isl_pw_aff *pwaff1,
        if (!pwaff1 || !pwaff2)
                goto error;
 
-       ctx = isl_dim_get_ctx(pwaff1->dim);
-       if (!isl_dim_equal(pwaff1->dim, pwaff2->dim))
+       ctx = isl_space_get_ctx(pwaff1->dim);
+       if (!isl_space_is_equal(pwaff1->dim, pwaff2->dim))
                isl_die(ctx, isl_error_invalid,
                        "arguments should live in same space", goto error);
 
@@ -1168,28 +1268,28 @@ static __isl_give isl_pw_aff *pw_aff_union_max(__isl_take isl_pw_aff *pwaff1,
        }
 
        n = 2 * (pwaff1->n + 1) * (pwaff2->n + 1);
-       res = isl_pw_aff_alloc_(isl_dim_copy(pwaff1->dim), n);
+       res = isl_pw_aff_alloc_size(isl_space_copy(pwaff1->dim), n);
 
        for (i = 0; i < pwaff1->n; ++i) {
                set = isl_set_copy(pwaff1->p[i].set);
                for (j = 0; j < pwaff2->n; ++j) {
                        struct isl_set *common;
-                       isl_set *ge;
+                       isl_set *better;
 
                        common = isl_set_intersect(
                                        isl_set_copy(pwaff1->p[i].set),
                                        isl_set_copy(pwaff2->p[j].set));
-                       ge = isl_set_from_basic_set(isl_aff_ge_basic_set(
+                       better = isl_set_from_basic_set(cmp(
                                        isl_aff_copy(pwaff2->p[j].aff),
                                        isl_aff_copy(pwaff1->p[i].aff)));
-                       ge = isl_set_intersect(common, ge);
-                       if (isl_set_plain_is_empty(ge)) {
-                               isl_set_free(ge);
+                       better = isl_set_intersect(common, better);
+                       if (isl_set_plain_is_empty(better)) {
+                               isl_set_free(better);
                                continue;
                        }
-                       set = isl_set_subtract(set, isl_set_copy(ge));
+                       set = isl_set_subtract(set, isl_set_copy(better));
 
-                       res = isl_pw_aff_add_piece(res, ge,
+                       res = isl_pw_aff_add_piece(res, better,
                                                isl_aff_copy(pwaff2->p[j].aff));
                }
                res = isl_pw_aff_add_piece(res, set,
@@ -1215,27 +1315,66 @@ error:
        return NULL;
 }
 
+/* Compute a piecewise quasi-affine expression with a domain that
+ * is the union of those of pwaff1 and pwaff2 and such that on each
+ * cell, the quasi-affine expression is the maximum of those of pwaff1
+ * and pwaff2.  If only one of pwaff1 or pwaff2 is defined on a given
+ * cell, then the associated expression is the defined one.
+ */
+static __isl_give isl_pw_aff *pw_aff_union_max(__isl_take isl_pw_aff *pwaff1,
+       __isl_take isl_pw_aff *pwaff2)
+{
+       return pw_aff_union_opt(pwaff1, pwaff2, &isl_aff_ge_basic_set);
+}
+
 __isl_give isl_pw_aff *isl_pw_aff_union_max(__isl_take isl_pw_aff *pwaff1,
        __isl_take isl_pw_aff *pwaff2)
 {
-       return align_params_pw_pw_and(pwaff1, pwaff2, &pw_aff_union_max);
+       return isl_pw_aff_align_params_pw_pw_and(pwaff1, pwaff2,
+                                                       &pw_aff_union_max);
+}
+
+/* Compute a piecewise quasi-affine expression with a domain that
+ * is the union of those of pwaff1 and pwaff2 and such that on each
+ * cell, the quasi-affine expression is the minimum of those of pwaff1
+ * and pwaff2.  If only one of pwaff1 or pwaff2 is defined on a given
+ * cell, then the associated expression is the defined one.
+ */
+static __isl_give isl_pw_aff *pw_aff_union_min(__isl_take isl_pw_aff *pwaff1,
+       __isl_take isl_pw_aff *pwaff2)
+{
+       return pw_aff_union_opt(pwaff1, pwaff2, &isl_aff_le_basic_set);
+}
+
+__isl_give isl_pw_aff *isl_pw_aff_union_min(__isl_take isl_pw_aff *pwaff1,
+       __isl_take isl_pw_aff *pwaff2)
+{
+       return isl_pw_aff_align_params_pw_pw_and(pwaff1, pwaff2,
+                                                       &pw_aff_union_min);
+}
+
+__isl_give isl_pw_aff *isl_pw_aff_union_opt(__isl_take isl_pw_aff *pwaff1,
+       __isl_take isl_pw_aff *pwaff2, int max)
+{
+       if (max)
+               return isl_pw_aff_union_max(pwaff1, pwaff2);
+       else
+               return isl_pw_aff_union_min(pwaff1, pwaff2);
 }
 
 /* Construct a map with as domain the domain of pwaff and
  * one-dimensional range corresponding to the affine expressions.
  */
-__isl_give isl_map *isl_map_from_pw_aff(__isl_take isl_pw_aff *pwaff)
+static __isl_give isl_map *map_from_pw_aff(__isl_take isl_pw_aff *pwaff)
 {
        int i;
-       isl_dim *dim;
+       isl_space *dim;
        isl_map *map;
 
        if (!pwaff)
                return NULL;
 
-       dim = isl_pw_aff_get_dim(pwaff);
-       dim = isl_dim_from_domain(dim);
-       dim = isl_dim_add(dim, isl_dim_out, 1);
+       dim = isl_pw_aff_get_space(pwaff);
        map = isl_map_empty(dim);
 
        for (i = 0; i < pwaff->n; ++i) {
@@ -1254,6 +1393,31 @@ __isl_give isl_map *isl_map_from_pw_aff(__isl_take isl_pw_aff *pwaff)
        return map;
 }
 
+/* Construct a map with as domain the domain of pwaff and
+ * one-dimensional range corresponding to the affine expressions.
+ */
+__isl_give isl_map *isl_map_from_pw_aff(__isl_take isl_pw_aff *pwaff)
+{
+       if (isl_space_is_set(pwaff->dim))
+               isl_die(isl_pw_aff_get_ctx(pwaff), isl_error_invalid,
+                       "space of input is not a map",
+                       return isl_pw_aff_free(pwaff));
+       return map_from_pw_aff(pwaff);
+}
+
+/* Construct a one-dimensional set with as parameter domain
+ * the domain of pwaff and the single set dimension
+ * corresponding to the affine expressions.
+ */
+__isl_give isl_set *isl_set_from_pw_aff(__isl_take isl_pw_aff *pwaff)
+{
+       if (!isl_space_is_set(pwaff->dim))
+               isl_die(isl_pw_aff_get_ctx(pwaff), isl_error_invalid,
+                       "space of input is not a set",
+                       return isl_pw_aff_free(pwaff));
+       return map_from_pw_aff(pwaff);
+}
+
 /* Return a set containing those elements in the domain
  * of pwaff where it is non-negative.
  */
@@ -1265,7 +1429,7 @@ __isl_give isl_set *isl_pw_aff_nonneg_set(__isl_take isl_pw_aff *pwaff)
        if (!pwaff)
                return NULL;
 
-       set = isl_set_empty(isl_pw_aff_get_dim(pwaff));
+       set = isl_set_empty(isl_pw_aff_get_domain_space(pwaff));
 
        for (i = 0; i < pwaff->n; ++i) {
                isl_basic_set *bset;
@@ -1293,7 +1457,7 @@ __isl_give isl_set *isl_pw_aff_zero_set(__isl_take isl_pw_aff *pwaff)
        if (!pwaff)
                return NULL;
 
-       set = isl_set_empty(isl_pw_aff_get_dim(pwaff));
+       set = isl_set_empty(isl_pw_aff_get_domain_space(pwaff));
 
        for (i = 0; i < pwaff->n; ++i) {
                isl_basic_set *bset;
@@ -1340,9 +1504,9 @@ static __isl_give isl_set *pw_aff_gte_set(__isl_take isl_pw_aff *pwaff1,
        pwaff1 = isl_pw_aff_add(pwaff1, isl_pw_aff_neg(pwaff2));
 
        if (strict) {
-               isl_dim *dim = isl_set_get_dim(set1);
+               isl_space *dim = isl_set_get_space(set1);
                isl_aff *aff;
-               aff = isl_aff_zero(isl_local_space_from_dim(dim));
+               aff = isl_aff_zero_on_domain(isl_local_space_from_space(dim));
                aff = isl_aff_add_constant_si(aff, -1);
                pwaff1 = isl_pw_aff_add(pwaff1, isl_pw_aff_alloc(set1, aff));
        } else
@@ -1431,7 +1595,7 @@ static __isl_give isl_set *pw_aff_list_set(__isl_take isl_pw_aff_list *list1,
                isl_die(ctx, isl_error_invalid,
                        "list should contain at least one element", goto error);
 
-       set = isl_set_universe(isl_pw_aff_get_dim(list1->p[0]));
+       set = isl_set_universe(isl_pw_aff_get_domain_space(list1->p[0]));
        for (i = 0; i < list1->n; ++i)
                for (j = 0; j < list2->n; ++j) {
                        isl_set *set_ij;
@@ -1657,7 +1821,7 @@ static __isl_give isl_pw_aff *pw_aff_mul(__isl_take isl_pw_aff *pwaff1,
                goto error;
 
        n = pwaff1->n * pwaff2->n;
-       res = isl_pw_aff_alloc_(isl_dim_copy(pwaff1->dim), n);
+       res = isl_pw_aff_alloc_size(isl_space_copy(pwaff1->dim), n);
 
        for (i = 0; i < pwaff1->n; ++i) {
                for (j = 0; j < pwaff2->n; ++j) {
@@ -1690,7 +1854,7 @@ error:
 __isl_give isl_pw_aff *isl_pw_aff_mul(__isl_take isl_pw_aff *pwaff1,
        __isl_take isl_pw_aff *pwaff2)
 {
-       return align_params_pw_pw_and(pwaff1, pwaff2, &pw_aff_mul);
+       return isl_pw_aff_align_params_pw_pw_and(pwaff1, pwaff2, &pw_aff_mul);
 }
 
 static __isl_give isl_pw_aff *pw_aff_min(__isl_take isl_pw_aff *pwaff1,
@@ -1706,7 +1870,7 @@ static __isl_give isl_pw_aff *pw_aff_min(__isl_take isl_pw_aff *pwaff1,
 __isl_give isl_pw_aff *isl_pw_aff_min(__isl_take isl_pw_aff *pwaff1,
        __isl_take isl_pw_aff *pwaff2)
 {
-       return align_params_pw_pw_and(pwaff1, pwaff2, &pw_aff_min);
+       return isl_pw_aff_align_params_pw_pw_and(pwaff1, pwaff2, &pw_aff_min);
 }
 
 static __isl_give isl_pw_aff *pw_aff_max(__isl_take isl_pw_aff *pwaff1,
@@ -1722,7 +1886,7 @@ static __isl_give isl_pw_aff *pw_aff_max(__isl_take isl_pw_aff *pwaff1,
 __isl_give isl_pw_aff *isl_pw_aff_max(__isl_take isl_pw_aff *pwaff1,
        __isl_take isl_pw_aff *pwaff2)
 {
-       return align_params_pw_pw_and(pwaff1, pwaff2, &pw_aff_max);
+       return isl_pw_aff_align_params_pw_pw_and(pwaff1, pwaff2, &pw_aff_max);
 }
 
 static __isl_give isl_pw_aff *pw_aff_list_reduce(