Add -pedantic to Makefile.standalone to match production.

src/IceDefs.h

Index: src/IceDefs.h
diff --git a/src/IceDefs.h b/src/IceDefs.h
index 55e09acb8681ba438b6cf6ad214a5f101b62c0c3..2f945f82513d5d00793922dd4594e3de2964466a 100644
--- a/src/IceDefs.h
+++ b/src/IceDefs.h
@@ -129,8 +129,14 @@ typedef uint32_t TimerIdT;
 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 TOKENPASTE(x, y) x##y
+#define TOKENPASTE2(x, y) TOKENPASTE(x, y)
 #define ICE_CACHELINE_BOUNDARY                                                 \
-  alignas(MaxCacheLineSize) struct {}
+  void TOKENPASTE2(FakeUsage, __LINE__)() {                                    \
+    (void) TOKENPASTE2(Boundary, __LINE__);                                    \
+  }                                                                            \
+  alignas(MaxCacheLineSize) struct TOKENPASTE2(FakeType, __LINE__) {           \
+  } TOKENPASTE2(Boundary, __LINE__)

[JF, 2015/01/28 01:28:45]

You don't need the fake usage as long as you instantiate the struct.

But all of this is in vain because the struct must occupy some storage, it isn't
zero-sized... So we're back to:

  __attribute__((aligned(Align))) int : 0;

[Jim Stichnoth, 2015/01/28 03:26:37]

Done.

[JF, 2015/01/28 07:14:45]

Richard makes a good point that you can wrap each field you want to be together
in an anonymous struct, and make that struct alignas.

That would be standard-friendly and more readable.

[Jim Stichnoth, 2015/01/28 15:39:18]

Yeah, that is pretty nice and clear.  Except is there a good way to initialize
the fields that are now wrapped inside an anonymous struct?  E.g. this
conversion from MyClass1 to MyClass2:

class MyClass1 {
  std::mutex MyFieldLock;
  MyType MyField;
public:
  MyClass1(ArgType Arg) : MyField(Arg) {}
};

class MyClass2 {
  alignas(64) struct {
    std::mutex MyFieldLock;
    MyType MyField;
  } F;
public:
  // This doesn't work:
  // MyClass2(ArgType Arg) : F.MyField(Arg) {}
  // This may work, depending on MyType and ArgType:
  MyClass2(ArgType Arg) { F.MyField = Arg; }
};

The other issue is that if you have N fields needing cache alignment, you
generally want N+1 instances of alignas(), so that fields after the explicitly
aligned ones start on a new cache line.  So even with the aligned (anonymous)
struct solution, you still need one more boundary marker.

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