isl_basic_set_opt: avoid invalid access on error path
[platform/upstream/isl.git] / isl_sample.c
index f4b2ea1..c6a0f31 100644 (file)
@@ -1,12 +1,25 @@
+/*
+ * Copyright 2008-2009 Katholieke Universiteit Leuven
+ *
+ * Use of this software is governed by the MIT license
+ *
+ * Written by Sven Verdoolaege, K.U.Leuven, Departement
+ * Computerwetenschappen, Celestijnenlaan 200A, B-3001 Leuven, Belgium
+ */
+
+#include <isl_ctx_private.h>
+#include <isl_map_private.h>
 #include "isl_sample.h"
 #include "isl_sample_piplib.h"
-#include "isl_vec.h"
-#include "isl_mat.h"
-#include "isl_seq.h"
-#include "isl_map_private.h"
+#include <isl/vec.h>
+#include <isl/mat.h>
+#include <isl/seq.h>
 #include "isl_equalities.h"
 #include "isl_tab.h"
 #include "isl_basis_reduction.h"
+#include <isl_factorization.h>
+#include <isl_point_private.h>
+#include <isl_options_private.h>
 
 static struct isl_vec *empty_sample(struct isl_basic_set *bset)
 {
@@ -45,12 +58,16 @@ static struct isl_vec *interval_sample(struct isl_basic_set *bset)
        bset = isl_basic_set_simplify(bset);
        if (!bset)
                return NULL;
-       if (isl_basic_set_fast_is_empty(bset))
+       if (isl_basic_set_plain_is_empty(bset))
                return empty_sample(bset);
        if (bset->n_eq == 0 && bset->n_ineq == 0)
                return zero_sample(bset);
 
        sample = isl_vec_alloc(bset->ctx, 2);
+       if (!sample)
+               goto error;
+       if (!bset)
+               return NULL;
        isl_int_set_si(sample->block.data[0], 1);
 
        if (bset->n_eq > 0) {
@@ -271,8 +288,8 @@ static struct isl_mat *tab_equalities(struct isl_tab *tab)
        if (!tab)
                return NULL;
 
-       isl_assert(tab->mat->ctx, tab->bset, return NULL);
-       bset = tab->bset;
+       bset = isl_tab_peek_bset(tab);
+       isl_assert(tab->mat->ctx, bset, return NULL);
 
        n_eq = tab->n_var - tab->n_col + tab->n_dead;
        if (tab->empty || n_eq == 0)
@@ -315,7 +332,8 @@ static struct isl_mat *initial_basis(struct isl_tab *tab)
        struct isl_mat *eq;
        struct isl_mat *Q;
 
-       n_eq = tab->n_var - tab->n_col + tab->n_dead;
+       tab->n_unbounded = 0;
+       tab->n_zero = n_eq = tab->n_var - tab->n_col + tab->n_dead;
        if (tab->empty || n_eq == 0 || n_eq == tab->n_var)
                return isl_mat_identity(tab->mat->ctx, 1 + tab->n_var);
 
@@ -329,6 +347,105 @@ static struct isl_mat *initial_basis(struct isl_tab *tab)
        return Q;
 }
 
+/* Compute the minimum of the current ("level") basis row over "tab"
+ * and store the result in position "level" of "min".
+ */
+static enum isl_lp_result compute_min(isl_ctx *ctx, struct isl_tab *tab,
+       __isl_keep isl_vec *min, int level)
+{
+       return isl_tab_min(tab, tab->basis->row[1 + level],
+                           ctx->one, &min->el[level], NULL, 0);
+}
+
+/* Compute the maximum of the current ("level") basis row over "tab"
+ * and store the result in position "level" of "max".
+ */
+static enum isl_lp_result compute_max(isl_ctx *ctx, struct isl_tab *tab,
+       __isl_keep isl_vec *max, int level)
+{
+       enum isl_lp_result res;
+       unsigned dim = tab->n_var;
+
+       isl_seq_neg(tab->basis->row[1 + level] + 1,
+                   tab->basis->row[1 + level] + 1, dim);
+       res = isl_tab_min(tab, tab->basis->row[1 + level],
+                   ctx->one, &max->el[level], NULL, 0);
+       isl_seq_neg(tab->basis->row[1 + level] + 1,
+                   tab->basis->row[1 + level] + 1, dim);
+       isl_int_neg(max->el[level], max->el[level]);
+
+       return res;
+}
+
+/* Perform a greedy search for an integer point in the set represented
+ * by "tab", given that the minimal rational value (rounded up to the
+ * nearest integer) at "level" is smaller than the maximal rational
+ * value (rounded down to the nearest integer).
+ *
+ * Return 1 if we have found an integer point (if tab->n_unbounded > 0
+ * then we may have only found integer values for the bounded dimensions
+ * and it is the responsibility of the caller to extend this solution
+ * to the unbounded dimensions).
+ * Return 0 if greedy search did not result in a solution.
+ * Return -1 if some error occurred.
+ *
+ * We assign a value half-way between the minimum and the maximum
+ * to the current dimension and check if the minimal value of the
+ * next dimension is still smaller than (or equal) to the maximal value.
+ * We continue this process until either
+ * - the minimal value (rounded up) is greater than the maximal value
+ *     (rounded down).  In this case, greedy search has failed.
+ * - we have exhausted all bounded dimensions, meaning that we have
+ *     found a solution.
+ * - the sample value of the tableau is integral.
+ * - some error has occurred.
+ */
+static int greedy_search(isl_ctx *ctx, struct isl_tab *tab,
+       __isl_keep isl_vec *min, __isl_keep isl_vec *max, int level)
+{
+       struct isl_tab_undo *snap;
+       enum isl_lp_result res;
+
+       snap = isl_tab_snap(tab);
+
+       do {
+               isl_int_add(tab->basis->row[1 + level][0],
+                           min->el[level], max->el[level]);
+               isl_int_fdiv_q_ui(tab->basis->row[1 + level][0],
+                           tab->basis->row[1 + level][0], 2);
+               isl_int_neg(tab->basis->row[1 + level][0],
+                           tab->basis->row[1 + level][0]);
+               if (isl_tab_add_valid_eq(tab, tab->basis->row[1 + level]) < 0)
+                       return -1;
+               isl_int_set_si(tab->basis->row[1 + level][0], 0);
+
+               if (++level >= tab->n_var - tab->n_unbounded)
+                       return 1;
+               if (isl_tab_sample_is_integer(tab))
+                       return 1;
+
+               res = compute_min(ctx, tab, min, level);
+               if (res == isl_lp_error)
+                       return -1;
+               if (res != isl_lp_ok)
+                       isl_die(ctx, isl_error_internal,
+                               "expecting bounded rational solution",
+                               return -1);
+               res = compute_max(ctx, tab, max, level);
+               if (res == isl_lp_error)
+                       return -1;
+               if (res != isl_lp_ok)
+                       isl_die(ctx, isl_error_internal,
+                               "expecting bounded rational solution",
+                               return -1);
+       } while (isl_int_le(min->el[level], max->el[level]));
+
+       if (isl_tab_rollback(tab, snap) < 0)
+               return -1;
+
+       return 0;
+}
+
 /* Given a tableau representing a set, find and return
  * an integer point in the set, if there is any.
  *
@@ -368,16 +485,19 @@ static struct isl_mat *initial_basis(struct isl_tab *tab)
  * basis vector.  "init" is true if we want the first value at the current
  * level and false if we want the next value.
  *
+ * At the start of each level, we first check if we can find a solution
+ * using greedy search.  If not, we continue with the exhaustive search.
+ *
  * The initial basis is the identity matrix.  If the range in some direction
  * contains more than one integer value, we perform basis reduction based
- * on the value of ctx->gbr
+ * on the value of ctx->opt->gbr
  *     - ISL_GBR_NEVER:        never perform basis reduction
  *     - ISL_GBR_ONCE:         only perform basis reduction the first
  *                             time such a range is encountered
  *     - ISL_GBR_ALWAYS:       always perform basis reduction when
  *                             such a range is encountered
  *
- * When ctx->gbr is set to ISL_GBR_ALWAYS, then we allow the basis
+ * When ctx->opt->gbr is set to ISL_GBR_ALWAYS, then we allow the basis
  * reduction computation to return early.  That is, as soon as it
  * finds a reasonable first direction.
  */ 
@@ -411,7 +531,7 @@ struct isl_vec *isl_tab_sample(struct isl_tab *tab)
 
        ctx = tab->mat->ctx;
        dim = tab->n_var;
-       gbr = ctx->gbr;
+       gbr = ctx->opt->gbr;
 
        if (tab->n_unbounded == tab->n_var) {
                sample = isl_tab_get_sample_value(tab);
@@ -437,42 +557,47 @@ struct isl_vec *isl_tab_sample(struct isl_tab *tab)
        reduced = 0;
 
        while (level >= 0) {
-               int empty = 0;
                if (init) {
-                       res = isl_tab_min(tab, tab->basis->row[1 + level],
-                                   ctx->one, &min->el[level], NULL, 0);
-                       if (res == isl_lp_empty)
-                               empty = 1;
-                       isl_assert(ctx, res != isl_lp_unbounded, goto error);
+                       int choice;
+
+                       res = compute_min(ctx, tab, min, level);
                        if (res == isl_lp_error)
                                goto error;
-                       if (!empty && isl_tab_sample_is_integer(tab))
+                       if (res != isl_lp_ok)
+                               isl_die(ctx, isl_error_internal,
+                                       "expecting bounded rational solution",
+                                       goto error);
+                       if (isl_tab_sample_is_integer(tab))
                                break;
-                       isl_seq_neg(tab->basis->row[1 + level] + 1,
-                                   tab->basis->row[1 + level] + 1, dim);
-                       res = isl_tab_min(tab, tab->basis->row[1 + level],
-                                   ctx->one, &max->el[level], NULL, 0);
-                       isl_seq_neg(tab->basis->row[1 + level] + 1,
-                                   tab->basis->row[1 + level] + 1, dim);
-                       isl_int_neg(max->el[level], max->el[level]);
-                       if (res == isl_lp_empty)
-                               empty = 1;
-                       isl_assert(ctx, res != isl_lp_unbounded, goto error);
+                       res = compute_max(ctx, tab, max, level);
                        if (res == isl_lp_error)
                                goto error;
-                       if (!empty && isl_tab_sample_is_integer(tab))
+                       if (res != isl_lp_ok)
+                               isl_die(ctx, isl_error_internal,
+                                       "expecting bounded rational solution",
+                                       goto error);
+                       if (isl_tab_sample_is_integer(tab))
                                break;
-                       if (!empty && !reduced && ctx->gbr != ISL_GBR_NEVER &&
-                           isl_int_lt(min->el[level], max->el[level])) {
+                       choice = isl_int_lt(min->el[level], max->el[level]);
+                       if (choice) {
+                               int g;
+                               g = greedy_search(ctx, tab, min, max, level);
+                               if (g < 0)
+                                       goto error;
+                               if (g)
+                                       break;
+                       }
+                       if (!reduced && choice &&
+                           ctx->opt->gbr != ISL_GBR_NEVER) {
                                unsigned gbr_only_first;
-                               if (ctx->gbr == ISL_GBR_ONCE)
-                                       ctx->gbr = ISL_GBR_NEVER;
+                               if (ctx->opt->gbr == ISL_GBR_ONCE)
+                                       ctx->opt->gbr = ISL_GBR_NEVER;
                                tab->n_zero = level;
-                               gbr_only_first = ctx->gbr_only_first;
-                               ctx->gbr_only_first =
-                                       ctx->gbr == ISL_GBR_ALWAYS;
+                               gbr_only_first = ctx->opt->gbr_only_first;
+                               ctx->opt->gbr_only_first =
+                                       ctx->opt->gbr == ISL_GBR_ALWAYS;
                                tab = isl_tab_compute_reduced_basis(tab);
-                               ctx->gbr_only_first = gbr_only_first;
+                               ctx->opt->gbr_only_first = gbr_only_first;
                                if (!tab || !tab->basis)
                                        goto error;
                                reduced = 1;
@@ -483,15 +608,17 @@ struct isl_vec *isl_tab_sample(struct isl_tab *tab)
                } else
                        isl_int_add_ui(min->el[level], min->el[level], 1);
 
-               if (empty || isl_int_gt(min->el[level], max->el[level])) {
+               if (isl_int_gt(min->el[level], max->el[level])) {
                        level--;
                        init = 0;
                        if (level >= 0)
-                               isl_tab_rollback(tab, snap[level]);
+                               if (isl_tab_rollback(tab, snap[level]) < 0)
+                                       goto error;
                        continue;
                }
                isl_int_neg(tab->basis->row[1 + level][0], min->el[level]);
-               tab = isl_tab_add_valid_eq(tab, tab->basis->row[1 + level]);
+               if (isl_tab_add_valid_eq(tab, tab->basis->row[1 + level]) < 0)
+                       goto error;
                isl_int_set_si(tab->basis->row[1 + level][0], 0);
                if (level + tab->n_unbounded < dim - 1) {
                        ++level;
@@ -515,19 +642,89 @@ struct isl_vec *isl_tab_sample(struct isl_tab *tab)
        } else
                sample = isl_vec_alloc(ctx, 0);
 
-       ctx->gbr = gbr;
+       ctx->opt->gbr = gbr;
        isl_vec_free(min);
        isl_vec_free(max);
        free(snap);
        return sample;
 error:
-       ctx->gbr = gbr;
+       ctx->opt->gbr = gbr;
        isl_vec_free(min);
        isl_vec_free(max);
        free(snap);
        return NULL;
 }
 
+static struct isl_vec *sample_bounded(struct isl_basic_set *bset);
+
+/* Compute a sample point of the given basic set, based on the given,
+ * non-trivial factorization.
+ */
+static __isl_give isl_vec *factored_sample(__isl_take isl_basic_set *bset,
+       __isl_take isl_factorizer *f)
+{
+       int i, n;
+       isl_vec *sample = NULL;
+       isl_ctx *ctx;
+       unsigned nparam;
+       unsigned nvar;
+
+       ctx = isl_basic_set_get_ctx(bset);
+       if (!ctx)
+               goto error;
+
+       nparam = isl_basic_set_dim(bset, isl_dim_param);
+       nvar = isl_basic_set_dim(bset, isl_dim_set);
+
+       sample = isl_vec_alloc(ctx, 1 + isl_basic_set_total_dim(bset));
+       if (!sample)
+               goto error;
+       isl_int_set_si(sample->el[0], 1);
+
+       bset = isl_morph_basic_set(isl_morph_copy(f->morph), bset);
+
+       for (i = 0, n = 0; i < f->n_group; ++i) {
+               isl_basic_set *bset_i;
+               isl_vec *sample_i;
+
+               bset_i = isl_basic_set_copy(bset);
+               bset_i = isl_basic_set_drop_constraints_involving(bset_i,
+                           nparam + n + f->len[i], nvar - n - f->len[i]);
+               bset_i = isl_basic_set_drop_constraints_involving(bset_i,
+                           nparam, n);
+               bset_i = isl_basic_set_drop(bset_i, isl_dim_set,
+                           n + f->len[i], nvar - n - f->len[i]);
+               bset_i = isl_basic_set_drop(bset_i, isl_dim_set, 0, n);
+
+               sample_i = sample_bounded(bset_i);
+               if (!sample_i)
+                       goto error;
+               if (sample_i->size == 0) {
+                       isl_basic_set_free(bset);
+                       isl_factorizer_free(f);
+                       isl_vec_free(sample);
+                       return sample_i;
+               }
+               isl_seq_cpy(sample->el + 1 + nparam + n,
+                           sample_i->el + 1, f->len[i]);
+               isl_vec_free(sample_i);
+
+               n += f->len[i];
+       }
+
+       f->morph = isl_morph_inverse(f->morph);
+       sample = isl_morph_vec(isl_morph_copy(f->morph), sample);
+
+       isl_basic_set_free(bset);
+       isl_factorizer_free(f);
+       return sample;
+error:
+       isl_basic_set_free(bset);
+       isl_factorizer_free(f);
+       isl_vec_free(sample);
+       return NULL;
+}
+
 /* Given a basic set that is known to be bounded, find and return
  * an integer point in the basic set, if there is any.
  *
@@ -541,11 +738,12 @@ static struct isl_vec *sample_bounded(struct isl_basic_set *bset)
        struct isl_ctx *ctx;
        struct isl_vec *sample;
        struct isl_tab *tab = NULL;
+       isl_factorizer *f;
 
        if (!bset)
                return NULL;
 
-       if (isl_basic_set_fast_is_empty(bset))
+       if (isl_basic_set_plain_is_empty(bset))
                return empty_sample(bset);
 
        dim = isl_basic_set_total_dim(bset);
@@ -556,15 +754,27 @@ static struct isl_vec *sample_bounded(struct isl_basic_set *bset)
        if (bset->n_eq > 0)
                return sample_eq(bset, sample_bounded);
 
+       f = isl_basic_set_factorizer(bset);
+       if (!f)
+               goto error;
+       if (f->n_group != 0)
+               return factored_sample(bset, f);
+       isl_factorizer_free(f);
+               
        ctx = bset->ctx;
 
-       tab = isl_tab_from_basic_set(bset);
-       if (!ISL_F_ISSET(bset, ISL_BASIC_SET_NO_IMPLICIT))
-               tab = isl_tab_detect_implicit_equalities(tab);
-       if (!tab)
-               goto error;
+       tab = isl_tab_from_basic_set(bset, 1);
+       if (tab && tab->empty) {
+               isl_tab_free(tab);
+               ISL_F_SET(bset, ISL_BASIC_SET_EMPTY);
+               sample = isl_vec_alloc(bset->ctx, 0);
+               isl_basic_set_free(bset);
+               return sample;
+       }
 
-       tab->bset = isl_basic_set_copy(bset);
+       if (!ISL_F_ISSET(bset, ISL_BASIC_SET_NO_IMPLICIT))
+               if (isl_tab_detect_implicit_equalities(tab) < 0)
+                       goto error;
 
        sample = isl_tab_sample(tab);
        if (!sample)
@@ -640,7 +850,7 @@ static struct isl_vec *rational_sample(struct isl_basic_set *bset)
        if (!bset)
                return NULL;
 
-       tab = isl_tab_from_basic_set(bset);
+       tab = isl_tab_from_basic_set(bset, 0);
        sample = isl_tab_get_sample_value(tab);
        isl_tab_free(tab);
 
@@ -697,7 +907,7 @@ static struct isl_basic_set *shift_cone(struct isl_basic_set *cone,
 
        total = isl_basic_set_total_dim(cone);
 
-       shift = isl_basic_set_alloc_dim(isl_basic_set_get_dim(cone),
+       shift = isl_basic_set_alloc_space(isl_basic_set_get_space(cone),
                                        0, 0, cone->n_ineq);
 
        for (i = 0; i < cone->n_ineq; ++i) {
@@ -757,7 +967,8 @@ static struct isl_vec *round_up_in_cone(struct isl_vec *vec,
 
        total = isl_basic_set_total_dim(cone);
        cone = isl_basic_set_preimage(cone, U);
-       cone = isl_basic_set_remove_dims(cone, 0, total - (vec->size - 1));
+       cone = isl_basic_set_remove_dims(cone, isl_dim_set,
+                                        0, total - (vec->size - 1));
 
        cone = shift_cone(cone, vec);
 
@@ -802,28 +1013,6 @@ error:
        return NULL;
 }
 
-/* Drop all constraints in bset that involve any of the dimensions
- * first to first+n-1.
- */
-static struct isl_basic_set *drop_constraints_involving
-       (struct isl_basic_set *bset, unsigned first, unsigned n)
-{
-       int i;
-
-       if (!bset)
-               return NULL;
-
-       bset = isl_basic_set_cow(bset);
-
-       for (i = bset->n_ineq - 1; i >= 0; --i) {
-               if (isl_seq_first_non_zero(bset->ineq[i] + 1 + first, n) == -1)
-                       continue;
-               isl_basic_set_drop_inequality(bset, i);
-       }
-
-       return bset;
-}
-
 /* Give a basic set "bset" with recession cone "cone", compute and
  * return an integer point in bset, if any.
  *
@@ -879,7 +1068,7 @@ __isl_give isl_vec *isl_basic_set_sample_with_cone(
        total = isl_basic_set_total_dim(cone);
        cone_dim = total - cone->n_eq;
 
-       M = isl_mat_sub_alloc(bset->ctx, cone->eq, 0, cone->n_eq, 1, total);
+       M = isl_mat_sub_alloc6(bset->ctx, cone->eq, 0, cone->n_eq, 1, total);
        M = isl_mat_left_hermite(M, 0, &U, NULL);
        if (!M)
                goto error;
@@ -889,7 +1078,8 @@ __isl_give isl_vec *isl_basic_set_sample_with_cone(
        bset = isl_basic_set_preimage(bset, isl_mat_copy(U));
 
        bounded = isl_basic_set_copy(bset);
-       bounded = drop_constraints_involving(bounded, total - cone_dim, cone_dim);
+       bounded = isl_basic_set_drop_constraints_involving(bounded,
+                                                  total - cone_dim, cone_dim);
        bounded = isl_basic_set_drop_dims(bounded, total - cone_dim, cone_dim);
        sample = sample_bounded(bounded);
        if (!sample || sample->size == 0) {
@@ -910,6 +1100,133 @@ error:
        return NULL;
 }
 
+static void vec_sum_of_neg(struct isl_vec *v, isl_int *s)
+{
+       int i;
+
+       isl_int_set_si(*s, 0);
+
+       for (i = 0; i < v->size; ++i)
+               if (isl_int_is_neg(v->el[i]))
+                       isl_int_add(*s, *s, v->el[i]);
+}
+
+/* Given a tableau "tab", a tableau "tab_cone" that corresponds
+ * to the recession cone and the inverse of a new basis U = inv(B),
+ * with the unbounded directions in B last,
+ * add constraints to "tab" that ensure any rational value
+ * in the unbounded directions can be rounded up to an integer value.
+ *
+ * The new basis is given by x' = B x, i.e., x = U x'.
+ * For any rational value of the last tab->n_unbounded coordinates
+ * in the update tableau, the value that is obtained by rounding
+ * up this value should be contained in the original tableau.
+ * For any constraint "a x + c >= 0", we therefore need to add
+ * a constraint "a x + c + s >= 0", with s the sum of all negative
+ * entries in the last elements of "a U".
+ *
+ * Since we are not interested in the first entries of any of the "a U",
+ * we first drop the columns of U that correpond to bounded directions.
+ */
+static int tab_shift_cone(struct isl_tab *tab,
+       struct isl_tab *tab_cone, struct isl_mat *U)
+{
+       int i;
+       isl_int v;
+       struct isl_basic_set *bset = NULL;
+
+       if (tab && tab->n_unbounded == 0) {
+               isl_mat_free(U);
+               return 0;
+       }
+       isl_int_init(v);
+       if (!tab || !tab_cone || !U)
+               goto error;
+       bset = isl_tab_peek_bset(tab_cone);
+       U = isl_mat_drop_cols(U, 0, tab->n_var - tab->n_unbounded);
+       for (i = 0; i < bset->n_ineq; ++i) {
+               int ok;
+               struct isl_vec *row = NULL;
+               if (isl_tab_is_equality(tab_cone, tab_cone->n_eq + i))
+                       continue;
+               row = isl_vec_alloc(bset->ctx, tab_cone->n_var);
+               if (!row)
+                       goto error;
+               isl_seq_cpy(row->el, bset->ineq[i] + 1, tab_cone->n_var);
+               row = isl_vec_mat_product(row, isl_mat_copy(U));
+               if (!row)
+                       goto error;
+               vec_sum_of_neg(row, &v);
+               isl_vec_free(row);
+               if (isl_int_is_zero(v))
+                       continue;
+               tab = isl_tab_extend(tab, 1);
+               isl_int_add(bset->ineq[i][0], bset->ineq[i][0], v);
+               ok = isl_tab_add_ineq(tab, bset->ineq[i]) >= 0;
+               isl_int_sub(bset->ineq[i][0], bset->ineq[i][0], v);
+               if (!ok)
+                       goto error;
+       }
+
+       isl_mat_free(U);
+       isl_int_clear(v);
+       return 0;
+error:
+       isl_mat_free(U);
+       isl_int_clear(v);
+       return -1;
+}
+
+/* Compute and return an initial basis for the possibly
+ * unbounded tableau "tab".  "tab_cone" is a tableau
+ * for the corresponding recession cone.
+ * Additionally, add constraints to "tab" that ensure
+ * that any rational value for the unbounded directions
+ * can be rounded up to an integer value.
+ *
+ * If the tableau is bounded, i.e., if the recession cone
+ * is zero-dimensional, then we just use inital_basis.
+ * Otherwise, we construct a basis whose first directions
+ * correspond to equalities, followed by bounded directions,
+ * i.e., equalities in the recession cone.
+ * The remaining directions are then unbounded.
+ */
+int isl_tab_set_initial_basis_with_cone(struct isl_tab *tab,
+       struct isl_tab *tab_cone)
+{
+       struct isl_mat *eq;
+       struct isl_mat *cone_eq;
+       struct isl_mat *U, *Q;
+
+       if (!tab || !tab_cone)
+               return -1;
+
+       if (tab_cone->n_col == tab_cone->n_dead) {
+               tab->basis = initial_basis(tab);
+               return tab->basis ? 0 : -1;
+       }
+
+       eq = tab_equalities(tab);
+       if (!eq)
+               return -1;
+       tab->n_zero = eq->n_row;
+       cone_eq = tab_equalities(tab_cone);
+       eq = isl_mat_concat(eq, cone_eq);
+       if (!eq)
+               return -1;
+       tab->n_unbounded = tab->n_var - (eq->n_row - tab->n_zero);
+       eq = isl_mat_left_hermite(eq, 0, &U, &Q);
+       if (!eq)
+               return -1;
+       isl_mat_free(eq);
+       tab->basis = isl_mat_lin_to_aff(Q);
+       if (tab_shift_cone(tab, tab_cone, U) < 0)
+               return -1;
+       if (!tab->basis)
+               return -1;
+       return 0;
+}
+
 /* Compute and return a sample point in bset using generalized basis
  * reduction.  We first check if the input set has a non-trivial
  * recession cone.  If so, we perform some extra preprocessing in
@@ -924,12 +1241,17 @@ static struct isl_vec *gbr_sample(struct isl_basic_set *bset)
        dim = isl_basic_set_total_dim(bset);
 
        cone = isl_basic_set_recession_cone(isl_basic_set_copy(bset));
+       if (!cone)
+               goto error;
 
        if (cone->n_eq < dim)
                return isl_basic_set_sample_with_cone(bset, cone);
 
        isl_basic_set_free(cone);
        return sample_bounded(bset);
+error:
+       isl_basic_set_free(bset);
+       return NULL;
 }
 
 static struct isl_vec *pip_sample(struct isl_basic_set *bset)
@@ -961,7 +1283,7 @@ static struct isl_vec *basic_set_sample(struct isl_basic_set *bset, int bounded)
                return NULL;
 
        ctx = bset->ctx;
-       if (isl_basic_set_fast_is_empty(bset))
+       if (isl_basic_set_plain_is_empty(bset))
                return empty_sample(bset);
 
        dim = isl_basic_set_n_dim(bset);
@@ -989,7 +1311,7 @@ static struct isl_vec *basic_set_sample(struct isl_basic_set *bset, int bounded)
        if (dim == 1)
                return interval_sample(bset);
 
-       switch (bset->ctx->ilp_solver) {
+       switch (bset->ctx->opt->ilp_solver) {
        case ISL_ILP_PIP:
                return pip_sample(bset);
        case ISL_ILP_GBR:
@@ -1039,7 +1361,7 @@ __isl_give isl_basic_set *isl_basic_set_from_vec(__isl_take isl_vec *vec)
                isl_int_neg(bset->eq[k][0], vec->el[1 + i]);
                isl_int_set(bset->eq[k][1 + i], vec->el[0]);
        }
-       isl_vec_free(vec);
+       bset->sample = vec;
 
        return bset;
 error:
@@ -1071,6 +1393,11 @@ error:
        return NULL;
 }
 
+__isl_give isl_basic_set *isl_basic_set_sample(__isl_take isl_basic_set *bset)
+{
+       return isl_basic_map_sample(bset);
+}
+
 __isl_give isl_basic_map *isl_map_sample(__isl_take isl_map *map)
 {
        int i;
@@ -1100,3 +1427,41 @@ __isl_give isl_basic_set *isl_set_sample(__isl_take isl_set *set)
 {
        return (isl_basic_set *) isl_map_sample((isl_map *)set);
 }
+
+__isl_give isl_point *isl_basic_set_sample_point(__isl_take isl_basic_set *bset)
+{
+       isl_vec *vec;
+       isl_space *dim;
+
+       dim = isl_basic_set_get_space(bset);
+       bset = isl_basic_set_underlying_set(bset);
+       vec = isl_basic_set_sample_vec(bset);
+
+       return isl_point_alloc(dim, vec);
+}
+
+__isl_give isl_point *isl_set_sample_point(__isl_take isl_set *set)
+{
+       int i;
+       isl_point *pnt;
+
+       if (!set)
+               return NULL;
+
+       for (i = 0; i < set->n; ++i) {
+               pnt = isl_basic_set_sample_point(isl_basic_set_copy(set->p[i]));
+               if (!pnt)
+                       goto error;
+               if (!isl_point_is_void(pnt))
+                       break;
+               isl_point_free(pnt);
+       }
+       if (i == set->n)
+               pnt = isl_point_void(isl_set_get_space(set));
+
+       isl_set_free(set);
+       return pnt;
+error:
+       isl_set_free(set);
+       return NULL;
+}