diff options
author | Francis Murtagh <francis.murtagh@arm.com> | 2018-08-30 17:18:37 +0100 |
---|---|---|
committer | Matthew Bentham <matthew.bentham@arm.com> | 2018-09-17 17:21:25 +0100 |
commit | 8c5e3dc9f04ac15aa926d1a1214610760830dbe2 (patch) | |
tree | 354518d6b2db9c46e4b698fcfbedb629a5569b48 /src/armnn/backends/RefWorkloads | |
parent | e7a86a4a3363993fb41b1ea62f23b3643b8b0c78 (diff) | |
download | armnn-8c5e3dc9f04ac15aa926d1a1214610760830dbe2.tar.gz |
IVGCVSW-1786 - Division by zero
* Added logic for different cases of division by zero for ref
* Added DivisionByZeroTest
* Updated SelectiveComparer to compare NAN and INFINITY
Change-Id: Iec6f89264b17a0b03fad5d7ec4b2dafc31fea5df
Diffstat (limited to 'src/armnn/backends/RefWorkloads')
-rw-r--r-- | src/armnn/backends/RefWorkloads/Division.cpp | 41 |
1 files changed, 39 insertions, 2 deletions
diff --git a/src/armnn/backends/RefWorkloads/Division.cpp b/src/armnn/backends/RefWorkloads/Division.cpp index 9837fea6b4..9c1142226c 100644 --- a/src/armnn/backends/RefWorkloads/Division.cpp +++ b/src/armnn/backends/RefWorkloads/Division.cpp @@ -8,6 +8,8 @@ #include <functional> +#include <cmath> + namespace { @@ -18,8 +20,43 @@ void ElementwiseDivision(unsigned int numElements, { for (unsigned int i = 0; i < numElements; ++i) { - //TODO How to handle divide by 0 - outData[i] = inData0[i] / inData1[i]; + if (inData1[i] != 0.0f) + { + outData[i] = inData0[i] / inData1[i]; + } + else if (inData0[i] == 0.0f) + { + if (!std::signbit(inData1[i])) + { + outData[i]= NAN; + } + else + { + outData[i]= -NAN; + } + } + else if (inData0[i] < 0.0f) + { + if (!std::signbit(inData1[i])) + { + outData[i] = -INFINITY; + } + else + { + outData[i] = INFINITY; + } + } + else + { + if (!std::signbit(inData1[i])) + { + outData[i] = INFINITY; + } + else + { + outData[i] = -INFINITY; + } + } } } |