This is an archive of the discontinued LLVM Phabricator instance.

"ErrorReturn checker" WIP review
AbandonedPublic

Authored by balazske on Sep 25 2020, 8:50 AM.

Details

Reviewers
Szelethus
Summary

This is a testing version for the checker that is implemented in
D72705. It is extended to make it usable on large projects.

This change it not intended to land, it is here only for
pre-evaluating the code.

Diff Detail

Event Timeline

balazske created this revision.Sep 25 2020, 8:50 AM
Herald added a project: Restricted Project. · View Herald Transcript
balazske requested review of this revision.Sep 25 2020, 8:50 AM

Here are the results with this checker on project 'emacs`.
Still more evaluation is needed.

Results for 'duckdb': link

Szelethus added inline comments.Sep 28 2020, 6:01 AM
clang/lib/StaticAnalyzer/Checkers/ErrorReturnChecker.cpp
505

This still causes more harm then good, I'd propose to remove it, even for evaluation/debug purposes.

balazske abandoned this revision.Jan 5 2021, 8:22 AM