Disable questionable code for handling isNullPointerConstant on value dependent
expression.
llvm-svn: 82217
diff --git a/clang/lib/AST/Expr.cpp b/clang/lib/AST/Expr.cpp
index d7565c7..60458b4 100644
--- a/clang/lib/AST/Expr.cpp
+++ b/clang/lib/AST/Expr.cpp
@@ -1627,8 +1627,8 @@
/// cast to void*.
bool Expr::isNullPointerConstant(ASTContext &Ctx) const {
// Ignore value dependent expressions.
- if (isValueDependent())
- return true;
+ assert(!isValueDependent() && "Unexpect value dependent expression!");
+
// Strip off a cast to void*, if it exists. Except in C++.
if (const ExplicitCastExpr *CE = dyn_cast<ExplicitCastExpr>(this)) {
if (!Ctx.getLangOptions().CPlusPlus) {
diff --git a/clang/test/SemaCXX/value-dependent-exprs.cpp b/clang/test/SemaCXX/value-dependent-exprs.cpp
index 603da9b..c70f895 100644
--- a/clang/test/SemaCXX/value-dependent-exprs.cpp
+++ b/clang/test/SemaCXX/value-dependent-exprs.cpp
@@ -14,10 +14,13 @@
int b[I];
};
+ // FIXME: I'm unclear where the right place to handle this is.
+#if 0
void f0(int *p) {
if (p == I) {
}
}
+#endif
#if 0
// FIXME: Not sure whether we care about these.