zaks.anna added a comment.

Sorry for the delay!!!



================
Comment at: 
include/clang/StaticAnalyzer/Core/PathSensitive/BasicValueFactory.h:180
+  inline const llvm::APSInt &getZeroWithTypeSize(QualType T,
+                                                 bool isUnsigned = true) {
+    return getValue(0, Ctx.getTypeSize(T), isUnsigned);
----------------
The isUnsigned parameter is not used and is not necessary as the type itself 
has this info.

Not clear if this function is expected to work for all types or only scalar 
types (maybe assert that the input is isScalarType()).


Repository:
  rL LLVM

https://reviews.llvm.org/D31029



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

Reply via email to