Skip to content

Commit

Permalink
Fix wrong environment test that breaks clang++ builds.
Browse files Browse the repository at this point in the history
g++ builds require extra flags rejected by clang++. The bug is that the
flags are actually added unconditionally. This commit fixes the
condition.

See #1474
  • Loading branch information
ic committed Apr 23, 2018
1 parent d3f6382 commit 54b048f
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions configure.ac
Original file line number Diff line number Diff line change
Expand Up @@ -135,8 +135,8 @@ fi
if $avx -o $avx2 -o $sse41; then
case "${host_os}" in
*darwin* | *-macos10*)
if test -z "$CLANG"; then
# When using AVX, AVX2, or SSE4.1:
if test g++ = "$CXX"; then
# When using AVX, AVX2, or SSE4.1 with g++:
# Must tell AS to use clang integrated assembler,
# instead of the GNU based system assembler.
CXXFLAGS="$CXXFLAGS -Wa,-q"
Expand Down

0 comments on commit 54b048f

Please sign in to comment.