NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.
Code looks good to me, but I'd wait for @george.karpenkov because it was his
idea.
================
Comment at:
lib/StaticAnalyzer/Checkers/UninitializedObject/UninitializedObjectChecker.cpp:502
+
+ auto AssertM = callExpr(callee(functionDecl(hasName("assert"))));
+ auto GuardM =
----------------
In a lot of standard libraries `assert()` is implemented as a macro. You might
want to catch the corresponding builtin or look at macro names (not sure if we
have a matcher for the latter but it should be easy to add).
https://reviews.llvm.org/D51866
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
