[analyzer] Add support for __builtin_constant_p to BuiltinFunctionChecker

[Felix Kostenzer via cfe-commits]

Hi,

Added evaluation of __builtin_constant_p to the dedicated StaticAnalyzer checker
along with a regression test.

'make clang-test' results:

Testing Time: 335.63s
  Expected Passes : 11769
  Expected Failures : 19
  Unsupported Tests : 50

- felix
Index: lib/StaticAnalyzer/Checkers/BuiltinFunctionChecker.cpp
===================================================================
--- lib/StaticAnalyzer/Checkers/BuiltinFunctionChecker.cpp
+++ lib/StaticAnalyzer/Checkers/BuiltinFunctionChecker.cpp
@@ -113,6 +113,23 @@ bool BuiltinFunctionChecker::evalCall(const CallExpr *CE,
     C.addTransition(state->BindExpr(CE, LCtx, V));
     return true;
   }
+  case Builtin::BI__builtin_constant_p: {
+    SVal V;
+    SValBuilder& SVB = C.getSValBuilder();
+
+    llvm::APSInt Result;
+    // Model semantics as 'A return of 0 does not indicate that the value is
+    // not a constant, but merely that GCC cannot prove it is a constant [...]'
+    if (CE->EvaluateAsInt(Result, C.getASTContext(), Expr::SE_NoSideEffects)) {
+      SVB.getBasicValueFactory().getAPSIntType(CE->getType()).apply(Result);
+      V = SVB.makeIntVal(Result);
+    }
+    else
+      V = SVB.makeZeroVal(CE->getType());
+
+    C.addTransition(state->BindExpr(CE, LCtx, V));
+    return true;
+  }
   }
 }
 
Index: test/Analysis/builtin-functions.cpp
===================================================================
--- test/Analysis/builtin-functions.cpp
+++ test/Analysis/builtin-functions.cpp
@@ -64,3 +64,18 @@ void g(int i) {
                                     // We give up the analysis on this path.
   }
 }
+
+void test_constant_p() {
+  int i = 1;
+  const int j = 2;
+  constexpr int k = 3;
+  clang_analyzer_eval(__builtin_constant_p(42)); // expected-warning {{TRUE}}
+  clang_analyzer_eval(__builtin_constant_p(i)); // expected-warning {{FALSE}}
+  clang_analyzer_eval(__builtin_constant_p(j)); // expected-warning {{TRUE}}
+  clang_analyzer_eval(__builtin_constant_p(k)); // expected-warning {{TRUE}}
+  clang_analyzer_eval(__builtin_constant_p(i + 42)); // expected-warning {{FALSE}}
+  clang_analyzer_eval(__builtin_constant_p(j + 42)); // expected-warning {{TRUE}}
+  clang_analyzer_eval(__builtin_constant_p(k + 42)); // expected-warning {{TRUE}}
+  clang_analyzer_eval(__builtin_constant_p(" ")); // expected-warning {{TRUE}}
+  clang_analyzer_eval(__builtin_constant_p(test_constant_p)); // expected-warning {{FALSE}}
+}

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits