Skip to content

build: suppress noisy warning#15260

Merged
opencv-pushbot merged 1 commit intoopencv:3.4from
tomoaki0705:suppressNoisyWarning
Aug 8, 2019
Merged

build: suppress noisy warning#15260
opencv-pushbot merged 1 commit intoopencv:3.4from
tomoaki0705:suppressNoisyWarning

Conversation

@tomoaki0705
Copy link
Copy Markdown
Contributor

@tomoaki0705 tomoaki0705 commented Aug 8, 2019 •
edited by alalek
Loading

Merge with contrib: opencv/opencv_contrib#2217

This pullrequest changes

suppress noisy warning outside of OpenCV

  • add -Wno-psabi when using GCC 6
  • add -Wundef for CUDA 10
  • add -Wdeprecated-declarations when using GCC 7
  • add -Wstrict-aliasing and -Wtautological-compare for GCC 7
  • replace cudaThreadSynchronize with cudaDeviceSynchronize ( ref )

Note that this function is deprecated because its name does not reflect its behavior. Its functionality is similar to the non-deprecated function cudaDeviceSynchronize(), which should be used instead.

/home/pi/opencv-fork/3rdparty/openexr/Half/half.cpp: In function ‘std::ostream& operator<<(std::ostream&, half)’:
/home/pi/opencv-fork/3rdparty/openexr/Half/half.cpp:225:1: note: parameter passing for argument of type ‘half’ will change in GCC 7.1
 operator << (ostream &os, half h)
 ^~~~~~~~

In file included from /usr/include/c++/6/bits/stl_algo.h:61:0,
                 from /usr/include/c++/6/algorithm:62,
                 from /home/pi/opencv-fork/3rdparty/openexr/IlmImf/ImfTileOffsets.cpp:47:
/usr/include/c++/6/bits/stl_heap.h: In function ‘void std::__adjust_heap(_RandomAccessIterator, _Distance, _Distance, _Tp, _Compare) [with _RandomAccessIterator = __gnu_cxx::__normal_iterator<Imf_opencv::{anonymous}::tilepos*, std::vector<Imf_opencv::{anonymous}::tilepos> >; _Distance = int; _Tp = Imf_opencv::{anonymous}::tilepos; _Compare = __gnu_cxx::__ops::_Iter_less_iter]’:
/usr/include/c++/6/bits/stl_heap.h:209:5: note: parameter passing for argument of type ‘__gnu_cxx::__normal_iterator<Imf_opencv::{anonymous}::tilepos*, std::vector<Imf_opencv::{anonymous}::tilepos> >’ will change in GCC 7.1
     __adjust_heap(_RandomAccessIterator __first, _Distance __holeIndex,
     ^~~~~~~~~~~~~

force_builders=Custom
buildworker:Custom=linux-4
docker_image:Custom=ubuntu-cuda:18.04

alalek
alalek reviewed Aug 8, 2019
View reviewed changes
Copy link
Copy Markdown
Member

@alalek alalek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the contribution!

@tomoaki0705
suppress noisy warning
40c71a2 
  * add -Wno-psabi when using GCC 6
  * add -Wundef for CUDA 10
  * add -Wdeprecated-declarations when using GCC 7
  * add -Wstrict-aliasing and -Wtautological-compare for GCC 7
  * replace cudaThreadSynchronize with cudaDeviceSynchronize
@tomoaki0705 tomoaki0705 force-pushed the suppressNoisyWarning branch from d9fb7ca to 40c71a2 Compare August 8, 2019 12:50
alalek
alalek approved these changes Aug 8, 2019
View reviewed changes
Copy link
Copy Markdown
Member

@alalek alalek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@opencv-pushbot opencv-pushbot merged commit 40c71a2 into opencv:3.4 Aug 8, 2019
opencv-pushbot pushed a commit that referenced this pull request Aug 8, 2019
@alalek
Merge pull request #15260 from tomoaki0705:suppressNoisyWarning
6f267be
@tomoaki0705 tomoaki0705 deleted the suppressNoisyWarning branch August 8, 2019 20:29
@alalek alalek mentioned this pull request Aug 13, 2019
Merged
@alalek alalek mentioned this pull request Aug 7, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alalek alalek alalek approved these changes

Assignees

@alalek alalek

Labels

category: build/install

Projects

None yet

Milestone

3.4.8

Development

Successfully merging this pull request may close these issues.

3 participants

@tomoaki0705 @alalek @opencv-pushbot