From: Matthew Fernandez <matthew.fernandez@gmail.com>
Date: Sat, 19 Sep 2020 16:56:58 +0000 (-0700)
Subject: don't squash -O2 when using ICC
X-Git-Tag: 2.46.0~20^2^2~65^2~2
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=8b4618d011ea9c2f5b8c466bc35acc467d03fd17;p=graphviz

don't squash -O2 when using ICC

ICC is Intel's C Compiler. All recent versions of it *do* support `-O2`.
---

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 2948e0961..428bb3185 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -13,6 +13,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Changed
 - Cgraph's agheap() API has been removed
 - Autotools build system support for eFence has been removed
+- Building Graphviz with ICC defaults to -O2 instead of -O0
 
 ### Fixed
 - gvpr: line numbers in gvpr errors/warnings are incorrect #1594
diff --git a/configure.ac b/configure.ac
index 9a400b01b..72308fca0 100644
--- a/configure.ac
+++ b/configure.ac
@@ -364,8 +364,6 @@ else
 	    CFLAGS=`echo ${CFLAGS} | sed 's/ -pipe//'`
 	    # icc doesn't like -ffast-math
 	    CFLAGS=`echo ${CFLAGS} | sed 's/ -ffast-math//'`
-	    # icc doesn't like -O2
-	    CFLAGS=`echo ${CFLAGS} | sed 's/ -O2/ -O0/'`
 	    ;;
 	* )
 	    case "${build}" in