diff options
| author |  Peter Hawkins <phawkins@google.com> | 2017-02-10 06:08:40 -0800 |
|---|---|---|
| committer |  TensorFlower Gardener <gardener@tensorflow.org> | 2017-02-10 06:26:24 -0800 |
| commit | 52fd3b960c85d5b299620fefe54c1146d615f8da (patch) | |
| tree | b79ca261c8f636b34859034701ba7fd5170682cc /tensorflow/stream_executor/lib | |
| parent | 468e2515301de568d47e84dc468c94ddd92ade81 (diff) | |
Remove problematic SE_RETURN_STATUS_AS_BOOL macro
The macro is only used in a handful of places and does not have any compelling reason to exist.
Change: 147147481
Diffstat (limited to 'tensorflow/stream_executor/lib')
| -rw-r--r-- | tensorflow/stream_executor/lib/status_macros.h | 16 |
1 files changed, 0 insertions, 16 deletions
diff --git a/tensorflow/stream_executor/lib/status_macros.h b/tensorflow/stream_executor/lib/status_macros.h index 6a8a11f083..816132e85a 100644 --- a/tensorflow/stream_executor/lib/status_macros.h +++ b/tensorflow/stream_executor/lib/status_macros.h @@ -50,20 +50,4 @@ limitations under the License. SE_ASSIGN_OR_RETURN_IMPL(__lhs, __rhs, \ SE_MACRO_CONCAT(__status_or_value, __COUNTER__)) -// Logs the status and returns false if it is in error; otherwise, returns true. -// -// The argument expression is guaranteed to be evaluated exactly once. -// -// TODO(leary) remove as many of these as possible with port::Status -// proliferation. -#define SE_RETURN_STATUS_AS_BOOL(__status) \ - do { \ - auto status = __status; \ - if (status.ok()) { \ - return true; \ - } \ - LOG(ERROR) << status; \ - return false; \ - } while (false) - #endif // TENSORFLOW_STREAM_EXECUTOR_LIB_STATUS_MACROS_H_ |