[cfe-commits] r41592 - in /cfe/trunk: AST/ASTContext.cpp AST/Type.cpp CodeGen/CodeGenTypes.cpp Sema/SemaDecl.cpp include/clang/AST/ASTContext.h include/clang/AST/Type.h

[Chris Lattner]

On Aug 29, 2007, at 6:06 PM, Steve Naroff wrote:
> To solve this, we now have 2 array types, ConstantArrayType and
> VariableArrayType. ConstantArrayType's are unique, VAT's aren't.
>

Wow, this is much nicer.  It's great when a bug report turns into a  
great IR cleanup :)

> +void ConstantArrayType::getAsStringInternal(std::string &S) const {
> +  S += '[';
> +  S += llvm::utostr_32(getSize().getZExtValue());
> +  S += ']';

This should use utostr, which handles 64-bit values.

Very nice Steve,

-Chris