faisalv added inline comments.
================ Comment at: lib/AST/ExprConstant.cpp:5061 + APValue RVal; + // FIXME: We need to make sure we're passing the right type that + // maintains cv-qualifiers. ---------------- I don't think we need this fixme - the type of the expression should be correct - all other const checks for mutability have already been performed, right? ================ Comment at: lib/AST/ExprConstant.cpp:5486 + } + //Info.FFDiag(E); + return false; ---------------- I need to delete this comment... Repository: rL LLVM https://reviews.llvm.org/D29748 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits