Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

unused -mf16c compiler flag during android build #15935

Closed
larroy opened this issue Aug 16, 2019 · 12 comments
Closed

unused -mf16c compiler flag during android build #15935

larroy opened this issue Aug 16, 2019 · 12 comments

Comments

@larroy
Copy link
Contributor

larroy commented Aug 16, 2019

ci/build.py -p android_armv7

[...]

[98/290] ccache /usr/arm-linux-androideabi/bin/arm-linux-androideabi-clang++  -DDMLC_USE_CXX11 -DDMLC_USE_CXX11=1 -DMSHADOW_INT64_TENSOR_SIZE=0 -DMSHADOW_IN_CXX11 -DMSHADOW_USE_CBLAS=1 -DMSHADOW_USE_CUDA=0 -DMSHADOW_USE_MKL=0 -DMSHADOW_USE_SSE=0 -DMXNET_USE_BLAS_OPEN=1 -DMXNET_USE_NCCL=0 -DMXNET_USE_OPENCV=0 -DMXNET_USE_SIGNAL_HANDLER=1 -DNDEBUG=1 -I/work/mxnet/include -I/work/mxnet/src -isystem /usr/arm-linux-androideabi/include -I/work/mxnet/3rdparty/mshadow -I/work/mxnet/3rdparty/nvidia_cub -I/work/mxnet/3rdparty/tvm/nnvm/include -I/work/mxnet/3rdparty/tvm/include -I/work/mxnet/3rdparty/dmlc-core/include -I/work/mxnet/3rdparty/dlpack/include -D_GLIBCXX_ASSERTIONS  -mf16c -Wall -Wno-unknown-pragmas -Wno-sign-compare -Wno-braced-scalar-init -O3 -g -std=c++11 -mf16c -std=c++0x -O2 -g -DNDEBUG -fPIC -MD -MT CMakeFiles/mxnet_static.dir/src/c_api/c_api_function.cc.o -MF CMakeFiles/mxnet_static.dir/src/c_api/c_api_function.cc.o.d -o CMakeFiles/mxnet_static.dir/src/c_api/c_api_function.cc.o -c /work/mxnet/src/c_api/c_api_function.cc
clang60++: warning: argument unused during compilation: '-mf16c' [-Wunused-command-line-argument]
clang60++: warning: argument unused during compilation: '-mf16c' [-Wunused-command-line-argument]
@mxnet-label-bot
Copy link
Contributor

Hey, this is the MXNet Label Bot.
Thank you for submitting the issue! I will try and suggest some labels so that the appropriate MXNet community members can help resolve it.
Here are my recommended labels: Build

@larroy
Copy link
Contributor Author

larroy commented Aug 16, 2019

@mxnet-label-bot add [Android, Build,Good First Issue]

@larroy
Copy link
Contributor Author

larroy commented Aug 16, 2019

@mxnet-label-bot add [Call for Contribution]

@michiboo
Copy link
Contributor

Hi is this issue still available?

@larroy
Copy link
Contributor Author

larroy commented Jan 25, 2020

@michiboo did you try to rebuild? ci/build.py -p android_armv7

@larroy
Copy link
Contributor Author

larroy commented Jan 25, 2020

Still happening @michiboo would you like to help?

@michiboo
Copy link
Contributor

@larroy Hi Yes I would like to still work on it

@larroy
Copy link
Contributor Author

larroy commented Jan 28, 2020

@michiboo thank you. Could you modify CMakeFile to omit these flags for the architectures where it doesn't apply?

@michiboo
Copy link
Contributor

michiboo commented Feb 1, 2020

@larroy Hi for some reason, I cannot reproduce the warning in this build

@larroy
Copy link
Contributor Author

larroy commented Feb 3, 2020

What did you try? ci/build.py -p android_armv7 should reproduce.

@michiboo
Copy link
Contributor

michiboo commented Feb 4, 2020

I try the exact same command, but I ended up with a error. I checked I already has opencv 4 so it shouldn't be a problem

  • cmake -DANDROID=ON -DUSE_CUDA=OFF -DUSE_SSE=OFF -DSUPPORT_F16C=OFF -DUSE_LAPACK=OFF -DUSE_OPENCV=OFF -DUSE_OPENMP=OFF -DUSE_SIGNAL_HANDLER=ON -DCMAKE_BUILD_TYPE=RelWithDebInfo -DUSE_MKL_IF_AVAILABLE=OFF -G Ninja /work/mxnet
    -- CMAKE_CROSSCOMPILING FALSE
    -- CMAKE_HOST_SYSTEM_PROCESSOR x86_64
    -- CMAKE_SYSTEM_PROCESSOR x86_64
    -- CMAKE_SYSTEM_NAME Linux
    -- CMake version '3.13.2' using generator 'Ninja'
    -- Determining F16C support
    -- F16C enabled
    -- Found OpenBLAS libraries: /usr/arm-linux-androideabi/lib/libopenblas.a
    -- Found OpenBLAS include: /usr/arm-linux-androideabi/include
    -- OpenCV Disabled
    CMake Error at CMakeLists.txt:491 (add_subdirectory):
    add_subdirectory given source "/work/mxnet/3rdparty/googletest/googletest"
    which is not an existing directory.

CMake Warning at CMakeLists.txt:847 (message):
OpenCV_VERSION_MAJOR: , version 3 with imgcodecs is required for im2rec,
im2rec will not be available

CMake Warning at example/image-classification/predict-cpp/CMakeLists.txt:3 (message):
OpenCV version >= 3 should be enabled and found to build image
classification example, skipping...

-- Configuring incomplete, errors occurred!
See also "/work/build/CMakeFiles/CMakeOutput.log".
2020-02-04 15:57:17,189 - root - INFO - Waiting for status of container d73f47c477fe for 600 s.
2020-02-04 15:57:17,303 - root - INFO - Container exit status: {'Error': None, 'StatusCode': 1}
2020-02-04 15:57:17,303 - root - ERROR - Container exited with an error 😞
2020-02-04 15:57:17,303 - root - INFO - Executed command for reproduction:

./ci/build.py -p android_armv7

2020-02-04 15:57:17,304 - root - INFO - Stopping container: d73f47c477fe
2020-02-04 15:57:17,305 - root - INFO - Removing container: d73f47c477fe
2020-02-04 15:57:17,322 - root - CRITICAL - Execution of ['/work/mxnet/ci/docker/runtime_functions.sh', 'build_android_armv7'] failed with status: 1

@leezu
Copy link
Contributor

leezu commented Feb 4, 2020

You need ot git submodule update --init --recursive. It seems your googletest submodule is missing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants