diff options
Diffstat (limited to 'eval/src')
-rw-r--r-- | eval/src/tests/eval/function/function_test.cpp | 14 | ||||
-rw-r--r-- | eval/src/vespa/eval/eval/llvm/llvm_wrapper.cpp | 3 | ||||
-rw-r--r-- | eval/src/vespa/eval/eval/test/tensor_conformance.cpp | 4 | ||||
-rw-r--r-- | eval/src/vespa/eval/eval/value.h | 2 | ||||
-rw-r--r-- | eval/src/vespa/eval/tensor/default_tensor_engine.cpp | 16 |
5 files changed, 11 insertions, 28 deletions
diff --git a/eval/src/tests/eval/function/function_test.cpp b/eval/src/tests/eval/function/function_test.cpp index 6c3839b6cc9..c2561a86d6f 100644 --- a/eval/src/tests/eval/function/function_test.cpp +++ b/eval/src/tests/eval/function/function_test.cpp @@ -14,20 +14,14 @@ std::vector<vespalib::string> params({"x", "y", "z", "w"}); double as_number(const Function &f) { auto number = as<Number>(f.root()); - if (number) { - return number->value(); - } else { - return error_value; - } + ASSERT_TRUE(number); + return number->value(); } vespalib::string as_string(const Function &f) { auto string = as<String>(f.root()); - if (string) { - return string->value(); - } else { - return "<error>"; - } + ASSERT_TRUE(string); + return string->value(); } struct OperatorLayer { diff --git a/eval/src/vespa/eval/eval/llvm/llvm_wrapper.cpp b/eval/src/vespa/eval/eval/llvm/llvm_wrapper.cpp index 71ca74f4167..cac24c6514f 100644 --- a/eval/src/vespa/eval/eval/llvm/llvm_wrapper.cpp +++ b/eval/src/vespa/eval/eval/llvm/llvm_wrapper.cpp @@ -17,6 +17,7 @@ #include <vespa/eval/eval/check_type.h> #include <vespa/vespalib/stllike/hash_set.h> #include <vespa/vespalib/util/approx.h> +#include <limits> double vespalib_eval_ldexp(double a, double b) { return std::ldexp(a, b); } double vespalib_eval_min(double a, double b) { return std::min(a, b); } @@ -318,7 +319,7 @@ struct FunctionBuilder : public NodeVisitor, public NodeTraverser { for (size_t i = 0; i < num_children; ++i) { discard(); } - push_double(error_value); + push_double(std::numeric_limits<double>::quiet_NaN()); } void make_call_1(llvm::Function *fun) { diff --git a/eval/src/vespa/eval/eval/test/tensor_conformance.cpp b/eval/src/vespa/eval/eval/test/tensor_conformance.cpp index 7e512bb5bf1..1e1bd828d41 100644 --- a/eval/src/vespa/eval/eval/test/tensor_conformance.cpp +++ b/eval/src/vespa/eval/eval/test/tensor_conformance.cpp @@ -38,8 +38,8 @@ struct Eval { double _number; TensorSpec _tensor; public: - Result() : _type(Type::ERROR), _number(error_value), _tensor("error") {} - Result(const TensorEngine &engine, const Value &value) : _type(Type::ERROR), _number(error_value), _tensor("error") { + Result() : _type(Type::ERROR), _number(0.0), _tensor("error") {} + Result(const TensorEngine &engine, const Value &value) : _type(Type::ERROR), _number(0.0), _tensor("error") { if (value.is_double()) { _type = Type::NUMBER; } diff --git a/eval/src/vespa/eval/eval/value.h b/eval/src/vespa/eval/eval/value.h index 15df44efbac..6701173bcd3 100644 --- a/eval/src/vespa/eval/eval/value.h +++ b/eval/src/vespa/eval/eval/value.h @@ -12,8 +12,6 @@ namespace eval { class Tensor; -constexpr double error_value = 31212.0; - /** * An abstract Value. **/ diff --git a/eval/src/vespa/eval/tensor/default_tensor_engine.cpp b/eval/src/vespa/eval/tensor/default_tensor_engine.cpp index a265ae5ae85..dc658d0b2da 100644 --- a/eval/src/vespa/eval/tensor/default_tensor_engine.cpp +++ b/eval/src/vespa/eval/tensor/default_tensor_engine.cpp @@ -81,9 +81,6 @@ const Value &to_default(const Value &value, Stash &stash) { } const Value &to_value(std::unique_ptr<Tensor> tensor, Stash &stash) { - if (!tensor) { - return stash.create<DoubleValue>(eval::error_value); - } if (tensor->type().is_tensor()) { return *stash.create<Value::UP>(std::move(tensor)); } @@ -91,9 +88,6 @@ const Value &to_value(std::unique_ptr<Tensor> tensor, Stash &stash) { } Value::UP to_value(std::unique_ptr<Tensor> tensor) { - if (!tensor) { - return std::make_unique<DoubleValue>(eval::error_value); - } if (tensor->type().is_tensor()) { return tensor; } @@ -343,13 +337,9 @@ DefaultTensorEngine::reduce(const Value &a, Aggr aggr, const std::vector<vespali return fallback_reduce(a, aggr, dimensions, stash); } } else { - if (dimensions.empty()) { - Aggregator &aggregator = Aggregator::create(aggr, stash); - aggregator.first(a.as_double()); - return stash.create<DoubleValue>(aggregator.result()); - } else { - return stash.create<DoubleValue>(eval::error_value); - } + Aggregator &aggregator = Aggregator::create(aggr, stash); + aggregator.first(a.as_double()); + return stash.create<DoubleValue>(aggregator.result()); } } |