[cfe-commits] [PATCH] PR12094: set alignment of memory builtins

Chris Lattner clattner at apple.com
Fri Mar 2 16:27:43 PST 2012


On Mar 2, 2012, at 1:39 PM, Jay Foad wrote:

> On 2 March 2012 20:09, Eli Friedman <eli.friedman at gmail.com> wrote:
>> I don't think this is really safe; strictly speaking, yes, an int* is
>> required to be appropriately aligned, but in practice neither clang
>> nor gcc has ever tried to enforce that, so making that assumption is
>> going to cause trouble.
> 
> I've had no complaints from buildbots in the last 3 hours. James
> Molloy has offered to run some tests on an ARM system that doesn't
> tolerate misaligned memory accesses. Can we leave this in for the time
> being and see just how much trouble it causes?

Eli and I chatted about this, and I think I convinced him.  It should stay in.  Thanks Jay!

-Chris



More information about the cfe-commits mailing list