[cfe-commits] r111338 - in /cfe/trunk: include/clang/AST/Type.h include/clang/Basic/DiagnosticSemaKinds.td lib/AST/Type.cpp lib/Sema/SemaDecl.cpp lib/Sema/SemaExprCXX.cpp lib/Sema/SemaType.cpp test/Sema/array-size-64.c test/Sema/array-size.c

[Eli Friedman]

On Tue, Aug 17, 2010 at 5:39 PM, Douglas Gregor <dgregor at apple.com> wrote:
> Author: dgregor
> Date: Tue Aug 17 19:39:00 2010
> New Revision: 111338
>
> URL: http://llvm.org/viewvc/llvm-project?rev=111338&view=rev
> Log:
> Emit an error if an array is too large. We're slightly more strict
> than GCC 4.2 here when building 32-bit (where GCC will allow
> allocation of an array for which we can't get a valid past-the-end
> pointer), and emulate its odd behavior in 64-bit where it only allows
> 63 bits worth of storage in the array. The former is a correctness
> issue; the latter is harmless in practice (you wouldn't be able to use
> such an array anyway) and helps us pass a GCC DejaGNU test.

Does this fix http://llvm.org/bugs/show_bug.cgi?id=1889?

-Eli