Removing unused Status helpers from pyiree.
diff --git a/bindings/python/pyiree/rt/status_utils.cc b/bindings/python/pyiree/rt/status_utils.cc index d629e73..a5bf054 100644 --- a/bindings/python/pyiree/rt/status_utils.cc +++ b/bindings/python/pyiree/rt/status_utils.cc
@@ -21,19 +21,6 @@ namespace { -PyObject* StatusToPyExcClass(const Status& status) { - switch (status.code()) { - case StatusCode::kInvalidArgument: - return PyExc_ValueError; - case StatusCode::kOutOfRange: - return PyExc_IndexError; - case StatusCode::kUnimplemented: - return PyExc_NotImplementedError; - default: - return PyExc_RuntimeError; - } -} - PyObject* ApiStatusToPyExcClass(iree_status_t status) { switch (iree_status_code(status)) { case IREE_STATUS_INVALID_ARGUMENT: @@ -49,12 +36,6 @@ } // namespace -pybind11::error_already_set StatusToPyExc(const Status& status) { - assert(!status.ok()); - PyErr_SetString(StatusToPyExcClass(status), status.ToString().c_str()); - return pybind11::error_already_set(); -} - pybind11::error_already_set ApiStatusToPyExc(iree_status_t status, const char* message) { assert(!iree_status_is_ok(status));
diff --git a/bindings/python/pyiree/rt/status_utils.h b/bindings/python/pyiree/rt/status_utils.h index a6a1ece..576aa4a 100644 --- a/bindings/python/pyiree/rt/status_utils.h +++ b/bindings/python/pyiree/rt/status_utils.h
@@ -16,20 +16,11 @@ #define IREE_BINDINGS_PYTHON_PYIREE_COMMON_STATUS_UTILS_H_ #include "iree/base/api.h" -#include "iree/base/status.h" #include "pybind11/pybind11.h" namespace iree { namespace python { -// Converts a failing status to a throwable exception, setting Python -// error information. -// Correct usage is something like: -// if (!status.ok()) { -// throw StatusToPyExc(status); -// } -pybind11::error_already_set StatusToPyExc(const Status& status); - // Raises a value error with the given message. // Correct usage: // throw RaiseValueError(PyExc_ValueError, "Foobar'd"); @@ -43,16 +34,6 @@ return RaisePyError(PyExc_ValueError, message); } -// Consumes a StatusOr<T>, returning an rvalue reference to the T if the -// status is ok(). Otherwise, throws an exception. -template <typename T> -T&& PyConsumeStatusOr(iree::StatusOr<T>&& sor) { - if (sor.ok()) { - return std::move(*sor); - } - throw StatusToPyExc(sor.status()); -} - pybind11::error_already_set ApiStatusToPyExc(iree_status_t status, const char* message);