This is an archive of the discontinued LLVM Phabricator instance.

[analyzer] Split up RetainCountChecker
ClosedPublic

Authored by george.karpenkov on Aug 15 2018, 5:06 PM.

Diff Detail

Event Timeline

NoQ added a comment.Aug 15 2018, 5:15 PM

I approve, it seems easy to understand where to look for things.

There's also ObjCRetainCount.h, would you like to merge or move it as well?

There's also ObjCRetainCount.h, would you like to merge or move it as well?

Apparently not:

lib/ARCMigrate/ObjCMT.cpp
30:#include "clang/StaticAnalyzer/Checkers/ObjCRetainCount.h"

NoQ accepted this revision.Aug 16 2018, 2:41 PM

Ugh. Maybe relocate it to lib/Analysis then?

This revision is now accepted and ready to land.Aug 16 2018, 2:41 PM
This revision was automatically updated to reflect the committed changes.