On Feb 17, 2008, at 11:13 PM, Anders Carlsson wrote: > + // FIXME: This is X86 specific, use 32-bit alignment for long > long. > + if (FD->getType()->isIntegerType() && FieldAlign > 32) > + FieldAlign = 32; I see this FIXE used more than once. It is a good idea to start using a TargetInfo hook now. - Devang