Unverified Commit 06bb90c7 authored by Matthew Brookhart's avatar Matthew Brookhart Committed by GitHub

fix clang specific warnings (#281)

parent f7a22b95
...@@ -13,14 +13,15 @@ ...@@ -13,14 +13,15 @@
limitations under the License. limitations under the License.
*/ */
#include "ngraph/builder/numpy_transpose.hpp"
#include <sstream> #include <sstream>
#include "ngraph/builder/numpy_transpose.hpp"
#include "ngraph/except.hpp" #include "ngraph/except.hpp"
#include "ngraph/ops/reshape.hpp" #include "ngraph/ops/reshape.hpp"
namespace ngraph namespace ngraph
{ {
void numpy_transpose_error(const AxisVector& order, const Shape& in_shape) [[noreturn]] void numpy_transpose_error(const AxisVector& order, const Shape& in_shape)
{ {
std::ostringstream os; std::ostringstream os;
os << "The axes order "; os << "The axes order ";
...@@ -49,7 +50,7 @@ namespace ngraph ...@@ -49,7 +50,7 @@ namespace ngraph
std::unordered_set<ngraph::AxisVector::value_type> axes; std::unordered_set<ngraph::AxisVector::value_type> axes;
for (auto o : order) for (auto o : order)
{ {
if (o >= 0 && o < in_shape.size() && !axes.count(o)) if (o < in_shape.size() && !axes.count(o))
{ {
axes.insert(o); axes.insert(o);
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment