On Aug 14, 2009, at 4:53 PM, Eli Friedman wrote: > On Fri, Aug 14, 2009 at 4:51 PM, Eli > Friedman<[email protected]> wrote: >> On Fri, Aug 14, 2009 at 4:43 PM, Chris Lattner<[email protected]> >> 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 _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
