Add -pedantic to Makefile.standalone to match production.

This also requires modifying the ICE_CACHELINE_BOUNDARY macro to avoid a warning about anonymous structs: src/IceUtils.h:132:3: warning: anonymous structs are a GNU extension [-Wgnu-anonymous-struct] BUG= none R=jfb@chromium.org Review URL: https://codereview.chromium.org/883983002

Add -pedantic to Makefile.standalone to match production.
26f9433e · Jim Stichnoth · 8427ea2b · 26f9433e · 26f9433e
Commit 26f9433e authored Jan 28, 2015 by Jim Stichnoth
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

Makefile.standalone Makefile.standalone +1 -1

IceDefs.h src/IceDefs.h +1 -1

No files found.
--- a/Makefile.standalone
+++ b/Makefile.standalone
@@ -92,7 +92,7 @@ CXX := CCACHE_CPP2=yes $(CCACHE) $(CLANG_PATH)/clang++

 CXXFLAGS := $(LLVM_CXXFLAGS) -std=c++11 -Wall -Wextra -Werror -fno-rtti \
 	-fno-exceptions $(OPTLEVEL) $(ASSERTIONS) $(CXX_DEFINES) -g \
-	$(HOST_FLAGS) -Wno-error=unused-parameter \
+	$(HOST_FLAGS) -pedantic -Wno-error=unused-parameter \
 	-I$(LIBCXX_INSTALL_PATH)/include/c++/v1 $(CXX_EXTRA)
 LDFLAGS := $(HOST_FLAGS) -L$(LIBCXX_INSTALL_PATH)/lib -Wl,--gc-sections \
 	$(LD_EXTRA)

--- a/src/IceDefs.h
+++ b/src/IceDefs.h
@@ -130,7 +130,7 @@ enum { MaxCacheLineSize = 64 };
 // Use ICE_CACHELINE_BOUNDARY to force the next field in a declaration
 // list to be aligned to the next cache line.
 #define ICE_CACHELINE_BOUNDARY                                                 \
-  alignas(MaxCacheLineSize) struct {}
+  __attribute__((aligned(MaxCacheLineSize))) int : 0

 // PNaCl is ILP32, so theoretically we should only need 32-bit offsets.
 typedef int32_t RelocOffsetT;