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

[James Molloy]

Hi,

Just to confirm, the only board I have available with the processing power
to build LLVM is Cortex-A9 based, and that doesn't trap misaligned loads.

I can build and run regression tests tomorrow - the board is currently busy
running the LLVM test suite.

Cheers,

James

-----Original Message-----
From: cfe-commits-bounces at cs.uiuc.edu
[mailto:cfe-commits-bounces at cs.uiuc.edu] On Behalf Of Jay Foad
Sent: 02 March 2012 21:40
To: Eli Friedman
Cc: cfe-commits at cs.uiuc.edu
Subject: Re: [cfe-commits] [PATCH] PR12094: set alignment of memory builtins

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?

Jay.
_______________________________________________
cfe-commits mailing list
cfe-commits at cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits