[cfe-commits] r146482 - /cfe/trunk/lib/AST/ASTContext.cpp
mclow.lists at gmail.com
Tue Dec 13 09:52:44 PST 2011
On Dec 13, 2011, at 9:37 AM, Peter Cooper wrote:
> Hi Abramo
> We're getting some buildbot failures on the gcc test suite since this change. The test has a very large union in it which must be triggering your new assert.
> Can you please have a look and see if your assert is correct on this test?
> The test in question is gcc.c-torture/execute/991014-1.c
A coding style suggestion (while you're at it).
you should use:
(and include <limits> if necessary)
>> + assert((Size == 0 || EltInfo.first <= (uint64_t)(-1)/Size) && "Overflow in array type bit size evaluation");
More information about the cfe-commits