> On Mon, Feb 2, 2009 at 7:00 AM, Nuno Lopes <[email protected]> wrote: >> - if (Res > llvm::APSInt(Res.getBitWidth(), Res.isUnsigned())) >> - return Context.getConstantArrayType(VLATy->getElementType(), >> - Res, ArrayType::Normal, 0); >> - return QualType(); >> + >> + return Context.getConstantArrayType(VLATy->getElementType(), >> + Res, ArrayType::Normal, 0); >> } > > We do not want to be letting though negative numbers here; instead of > eliminating the check completely, we should explicitly check for Res > == 0.
Ah, right, thanks. For some unknown reason I though the value had to be always unsigned. I'll fix it right away. Nuno _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
