Skip to content

Commit

Permalink
Compile with -Wstrict-overflow=1, not =5.
Browse files Browse the repository at this point in the history
When disabling ARCH_MIN_SSE2, the ANSI code path triggers several untraceable warnings that are nonsense (like integer union members giving warnings about pointer overflow).  GCC purportedly has raised false positives with the overflow detection setting this high.  Since it cannot give reliable reports in the ANSI code path for this RSP plugin (which does very little with what are actually pointers anyway in the vector unit), the level has been lowered down to 1.
  • Loading branch information
Iconoclast committed Mar 18, 2018
1 parent 0e27d9f commit b2fd3d3
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions make.sh
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ FLAGS_x86="\
-pedantic \
-Wall -Wshadow -Wredundant-decls -Wextra -Wcast-align -Wcast-qual \
-Wdisabled-optimization -Wformat=2 -Winit-self -Wlogical-op
-Wmissing-include-dirs -Wstrict-overflow=5 -Wundef -Wno-unused \
-Wmissing-include-dirs -Wstrict-overflow=1 -Wundef -Wno-unused \
-Wno-variadic-macros -Wno-parentheses -fdiagnostics-show-option"
else
FLAGS_x86="\
Expand All @@ -41,7 +41,7 @@ FLAGS_x86="\
-pedantic \
-Wall -Wshadow -Wredundant-decls -Wextra -Wcast-align -Wcast-qual \
-Wdisabled-optimization -Wformat=2 -Winit-self -Wlogical-op
-Wmissing-include-dirs -Wstrict-overflow=5 -Wundef -Wno-unused \
-Wmissing-include-dirs -Wstrict-overflow=1 -Wundef -Wno-unused \
-Wno-variadic-macros -Wno-parentheses -fdiagnostics-show-option"
fi
C_FLAGS=$FLAGS_x86 # default since Intel SIMD was the most tested
Expand Down

0 comments on commit b2fd3d3

Please sign in to comment.