[cfe-commits] r111794 - in /cfe/trunk: lib/CodeGen/CGExpr.cpp test/CodeGenCXX/x86_64-arguments.cpp

Chris Lattner clattner at apple.com
Mon Aug 23 10:24:06 PDT 2010


On Aug 23, 2010, at 7:00 AM, Douglas Gregor wrote:

> 
> On Aug 22, 2010, at 10:26 PM, Chris Lattner wrote:
> 
>> Author: lattner
>> Date: Mon Aug 23 00:26:13 2010
>> New Revision: 111794
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=111794&view=rev
>> Log:
>> fix rdar://8340348, a miscompile of boost that was exposed by r109848.
>> That revision started classifying truly empty structs like "Y" and "X"
>> as being NoClass/NoClass and turning them into 'ignore'.  The call code
>> turns around and allocates space for the ignored argument with 
>> GetUndefRValue.  The bug is that GetUndefRValue would return the address
>> as undef, instead of returning an object with a defined address but 
>> undefined contents.
> 
> Thanks! Boost.Graph is now passing all of its regression tests (again).

Woot, thanks for the nice reduced testcase!

-Chris



More information about the cfe-commits mailing list