Skip to content

Commit

Permalink
Merge pull request #2215 from jwillemsen/jwi-gcc42check
Browse files Browse the repository at this point in the history
Remove check for gcc 4.2, any compiler we support is newer
  • Loading branch information
jwillemsen committed Apr 11, 2024
2 parents e2056c2 + fc698ff commit 0ede814
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions ACE/include/makeinclude/platform_g++_common.GNU
Original file line number Diff line number Diff line change
Expand Up @@ -111,13 +111,11 @@ ifeq ($(no_strict_aliasing), 1)
FLAGS_C_CC += -fno-strict-aliasing
else
ifneq ($(no_strict_aliasing), 0)
# if not explicitly disabled enable suppression of strict-aliasing checks by default
# for GCC >= 4.2; these checks and the resulting warnings are very controversial and
# if not explicitly disabled enable suppression of strict-aliasing checks by default;
# these checks and the resulting warnings are very controversial and
# popular opinion on the web seems to be that it brings little practical value and a
# lot of pain to attempt to solve code issues and the best way to deal is to suppress
ifneq ($(findstring $(CXX_MAJOR_VERSION).$(CXX_MINOR_VERSION),4.1),$(CXX_MAJOR_VERSION).$(CXX_MINOR_VERSION))
FLAGS_C_CC += -fno-strict-aliasing
endif
FLAGS_C_CC += -fno-strict-aliasing
endif
endif

Expand Down

0 comments on commit 0ede814

Please sign in to comment.