Don't trigger assert in clang::ParmVarDecl::getDefaultArg

Change-Id: Ica385b513d87d639496ac470da296e871b093ab4

Don't trigger assert in clang::ParmVarDecl::getDefaultArg
Change-Id: Ica385b513d87d639496ac470da296e871b093ab4
65aa35fb · Stephan Bergmann · 9d094b9f · 65aa35fb
Kaydet (Commit) 65aa35fb authored Agu 14, 2015 tarafından Stephan Bergmann
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 2 deletions

defaultparams.cxx compilerplugins/clang/defaultparams.cxx +6 -2

No files found.
--- a/compilerplugins/clang/defaultparams.cxx
+++ b/compilerplugins/clang/defaultparams.cxx
@@ -52,9 +52,13 @@ bool DefaultParams::VisitCallExpr(CallExpr * callExpr) {
    if (isa<UnaryExprOrTypeTraitExpr>(arg))
        return true;
    const ParmVarDecl* parmVarDecl = functionDecl->getParamDecl(i);
+    if (!parmVarDecl->hasDefaultArg()
+        || parmVarDecl->hasUninstantiatedDefaultArg())
+    {
+        return true;
+    }
    const Expr* defaultArgExpr = parmVarDecl->getDefaultArg();
-    if (parmVarDecl->hasDefaultArg() &&
-        defaultArgExpr &&
+    if (defaultArgExpr &&
        defaultArgExpr->getType()->isIntegralType(compiler.getASTContext()))
    {
        APSInt x1, x2;