[cfe-commits] r79057 - /cfe/trunk/lib/CodeGen/CGExpr.cpp

[Chris Lattner]

On Aug 14, 2009, at 4:53 PM, Eli Friedman wrote:

> On Fri, Aug 14, 2009 at 4:51 PM, Eli  
> Friedman<eli.friedman at gmail.com> wrote:
>> On Fri, Aug 14, 2009 at 4:43 PM, Chris Lattner<sabre at nondot.org>  
>> wrote:
>>> Author: lattner
>>> Date: Fri Aug 14 18:43:22 2009
>>> New Revision: 79057
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?rev=79057&view=rev
>>> Log:
>>> use GetVLASize instead of accessing VLASizeMap directly, this gets  
>>> an assert if
>>> VLASize isn't populated for the type yet.
>>
>> Do you have a testcase?  This is just wallpapering over a real bug.
>
> Wait, nevermind; I just realized what GetVLASize does...

Right, there is no functionality change for a correct program.  Now  
clang asserts instead of crashing on http://llvm.org/bugs/show_bug.cgi?id=4721

-Chris