Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mkldnn_inner_product_layer.cpp:357:2: error: this ‘else’ #586

Closed
MyraBaba opened this issue May 9, 2018 · 4 comments

Comments

@MyraBaba
Copy link

@MyraBaba MyraBaba commented May 9, 2018

OS: ubuntu 17
CPU only

I have below error when compiling, any idea ?

38%] Building CXX object src/caffe/CMakeFiles/caffe.dir/layers/mkldnn_concat_layer.cpp.o
[ 38%] Building CXX object src/caffe/CMakeFiles/caffe.dir/layers/mkldnn_convolution_layer.cpp.o
[ 39%] Building CXX object src/caffe/CMakeFiles/caffe.dir/layers/mkldnn_eltwise_layer.cpp.o
[ 39%] Building CXX object src/caffe/CMakeFiles/caffe.dir/layers/mkldnn_inner_product_layer.cpp.o
[ 39%] Building CXX object src/caffe/CMakeFiles/caffe.dir/layers/mkldnn_lrn_layer.cpp.o
[ 41%] Building CXX object src/caffe/CMakeFiles/caffe.dir/layers/mkldnn_pooling_layer.cpp.o
/home/osboxes/openpose/3rdparty/caffe/src/caffe/layers/mkldnn_inner_product_layer.cpp: In member function ‘void caffe::MKLDNNInnerProductLayer::InitInnerProductBwd(const std::vector<caffe::Blob>&, const std::vector&, const std::vector<caffe::Blob>&)’:
/home/osboxes/openpose/3rdparty/caffe/src/caffe/layers/mkldnn_inner_product_layer.cpp:357:2: error: this ‘else’ clause does not guard... [-Werror=misleading-indentation]
else
^~~~
/home/osboxes/openpose/3rdparty/caffe/src/caffe/layers/mkldnn_inner_product_layer.cpp:361:5: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘else’
ipBwdData_desc.reset(new inner_product_backward_data::desc(init_bottom_md, init_weights_md, init_top_md));
^~~~~~~~~~~~~~

@peteruhrig

This comment has been minimized.

Copy link

@peteruhrig peteruhrig commented May 30, 2018

See here:
intel/caffe#202

@stale

This comment has been minimized.

Copy link

@stale stale bot commented Jul 29, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale/old label Jul 29, 2018
@stale stale bot closed this Aug 5, 2018
@peteruhrig

This comment has been minimized.

Copy link

@peteruhrig peteruhrig commented Oct 24, 2019

The problem is still there although it has been fixed in Intel Caffe.

@peteruhrig

This comment has been minimized.

Copy link

@peteruhrig peteruhrig commented Oct 25, 2019

To fix, add 3 spaces at the beginning of lines 354 and 357 and 4 spaces at the beginning of lines 355 and 358.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.