add GCC option -fno-strict-aliasing

see: twitter#276
yashsarma · Nov 14, 2014 · 1f17bc1 · 1f17bc1
1 parent 591b25e
commit 1f17bc1
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/README.md b/README.md
@@ -16,15 +16,21 @@ To build nutcracker from [distribution tarball](http://code.google.com/p/twempro
     $ make
     $ sudo make install
 
-To build nutcracker from source with _debug logs enabled_ and _assertions disabled_:
+To build nutcracker from source with _debug logs enabled_ and _assertions enabled_:
 
     $ git clone git@github.com:twitter/twemproxy.git
     $ cd twemproxy
     $ autoreconf -fvi
-    $ ./configure --enable-debug=log
+    $ ./configure --enable-debug=full
     $ make
     $ src/nutcracker -h
 
+Few checklists:
+
++ use newer version of gcc (older version of gcc has problems)
++ use CFLAGS="-O1" ./configure && make
++ use CFLAGS="-O3 -fno-strict-aliasing" ./configure && make
+
 ## Features
 
 + Fast.

diff --git a/src/Makefile.am b/src/Makefile.am
@@ -9,7 +9,9 @@ AM_CPPFLAGS += -I $(top_srcdir)/src/proto
 AM_CPPFLAGS += -I $(top_srcdir)/src/event
 AM_CPPFLAGS += -I $(top_srcdir)/contrib/yaml-0.1.4/include
 
-AM_CFLAGS =
+AM_CFLAGS = 
+# about -fno-strict-aliasing: https://github.com/twitter/twemproxy/issues/276
+AM_CFLAGS += -fno-strict-aliasing
 AM_CFLAGS += -Wall -Wshadow
 AM_CFLAGS += -Wpointer-arith
 AM_CFLAGS += -Winline