[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
clattner at apple.com
Thu Aug 30 11:26:35 PDT 2007
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
More information about the cfe-commits
mailing list