dcoughlin accepted this revision. dcoughlin added a comment. This revision is now accepted and ready to land.
Other than a suggested diagnostic rewording to consider, looks good to me. Thanks Gábor! ================ Comment at: lib/StaticAnalyzer/Checkers/PointerArithChecker.cpp:198 @@ +197,3 @@ + new BuiltinBug(this, "Dangerous pointer arithmetic", + "Pointer arithmetic done on non-array variables means " + "reliance on memory layout, which is dangerous.")); ---------------- Here is a suggestion to reword: "Pointer arithmetic on non-array variables relies on memory layout, which is dangerous." http://reviews.llvm.org/D14203 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
