[cfe-commits] r129673 - in /cfe/trunk: lib/CodeGen/CGRecordLayoutBuilder.cpp test/CodeGenCXX/class-layout.cpp test/CodeGenCXX/global-init.cpp test/CodeGenCXX/pointers-to-data-members.cpp test/CodeGenCXX/pragma-pack.cpp test/CodeGenCXX/x86_64-arguments.cpp

[Chris Lattner]

On Apr 17, 2011, at 2:56 PM, Anders Carlsson wrote:

> Author: andersca
> Date: Sun Apr 17 16:56:13 2011
> New Revision: 129673
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=129673&view=rev
> Log:
> When laying out bases in, always try the "base subobject" LLVM type. If it
> turns out that a field or base needs to be laid out in the tail padding of
> the base, CGRecordLayoutBuilder::ResizeLastBaseFieldIfNecessary will convert
> it to an array of i8.
> 
> I've audited the new test results to make sure that they are still valid. I've
> also verified that we pass a self-host with this change.
> 
> This (finally) fixes PR5589!

Woohoo!

-Chris