ROCm / MIVisionX

MIVisionX toolkit is a set of comprehensive computer vision and machine intelligence libraries, utilities, and applications bundled into a single toolkit. AMD MIVisionX also delivers a highly optimized open-source implementation of the Khronos OpenVX™ and OpenVX™ Extensions.
https://rocm.docs.amd.com/projects/MIVisionX/en/latest/
MIT License
186 stars 73 forks source link

rocAL - Warning on TOT Develop #816

Closed kiritigowda closed 2 years ago

kiritigowda commented 2 years ago

-- The C compiler identification is GNU 9.4.0 -- The CXX compiler identification is GNU 9.4.0

In file included from /home/kiriti/develop/docker/MIVisionX/rocAL/rocAL/source/master_graph.cpp:30:
/home/kiriti/develop/docker/MIVisionX/rocAL/rocAL/include/master_graph.h: In constructor ‘MasterGraph::MasterGraph(size_t, RaliAffinity, int, size_t, size_t, RaliTensorDataType)’:
/home/kiriti/develop/docker/MIVisionX/rocAL/rocAL/include/master_graph.h:185:20: warning: ‘MasterGraph::_box_encoder_gpu’ will be initialized after [-Wreorder]
  185 |     BoxEncoderGpu *_box_encoder_gpu;
      |                    ^~~~~~~~~~~~~~~~
/home/kiriti/develop/docker/MIVisionX/rocAL/rocAL/include/master_graph.h:160:15: warning:   ‘TimingDBG MasterGraph::_process_time’ [-Wreorder]
  160 |     TimingDBG _process_time, _bencode_time;
      |               ^~~~~~~~~~~~~
/home/kiriti/develop/docker/MIVisionX/rocAL/rocAL/source/master_graph.cpp:114:1: warning:   when initialized here [-Wreorder]
  114 | MasterGraph::MasterGraph(size_t batch_size, RaliAffinity affinity, int gpu_id, size_t cpu_threads, size_t prefetch_queue_depth, RaliTensorDataType output_tensor_data_type):
      | ^~~~~~~~~~~
LakshmiKumar23 commented 2 years ago

@swetha097 Can you look into this?

kiritigowda commented 2 years ago

@LakshmiKumar23 any updates on this?

rrawther commented 2 years ago

I saw this. It happened with the recent PR merge. I will fix it

kiritigowda commented 2 years ago

823

Scanning dependencies of target rali
[ 67%] Building CXX object rocAL/rocAL/CMakeFiles/rali.dir/source/master_graph.cpp.o
[ 68%] Building CXX object rocAL/rocAL/CMakeFiles/rali.dir/source/rali_api.cpp.o
[ 68%] Building CXX object rocAL/rocAL/CMakeFiles/rali.dir/source/rali_api_augmentation.cpp.o
[ 68%] Building CXX object rocAL/rocAL/CMakeFiles/rali.dir/source/rali_api_data_loaders.cpp.o
[ 68%] Building CXX object rocAL/rocAL/CMakeFiles/rali.dir/source/rali_api_data_transfer.cpp.o
[ 69%] Building CXX object rocAL/rocAL/CMakeFiles/rali.dir/source/rali_api_info.cpp.o
[ 69%] Building CXX object rocAL/rocAL/CMakeFiles/rali.dir/source/rali_api_meta_data.cpp.o
[ 69%] Linking CXX shared library ../../lib/librali.so
[ 99%] Built target rali
[100%] Built target rocAL_hip