[cfe-commits] r51673 - in /cfe/trunk: lib/CodeGen/CodeGenTypes.cpp test/CodeGen/struct-x86-darwin.c test/CodeGen/struct.c
dpatel at apple.com
Thu May 29 13:24:39 PDT 2008
On May 29, 2008, at 3:58 AM, Eli Friedman wrote:
> Author: efriedma
> Date: Thu May 29 05:58:49 2008
> New Revision: 51673
> URL: http://llvm.org/viewvc/llvm-project?rev=51673&view=rev
> Always use packed structs. This isn't really very nice, but there's
> nothing fundamentally wrong with it. Emitting unpacked structs where
> possible is more work for almost no practical benefit.
It may cause performance degradation in real world.
> We'll probably
> want to fix it at some point anyway, but it's low priority.
Please add FIXME in the code.
> The issue with long double in particular is that LLVM thinks an X86
> double is 10 bytes, while clang considers it for all purposes to be
> either 12 or 16 bytes, depending on the platform, even in a packed
I expect clang and LLVM to use same target data. Is it not true ?
More information about the cfe-commits