This is an archive of the discontinued LLVM Phabricator instance.

[analyzer] Add ExprMutationAnalyzer
ClosedPublic

Authored by shuaiwang on Sep 11 2018, 2:07 PM.

Details

Summary

This is 1/2 of moving ExprMutationAnalyzer from clangtidy to
clang/Analysis.
This diff along simply copies the ExprMutationAnalyzer over with trivial
modifications (e.g. include path, namespace)
2/2 will migrate existing usage of ExprMutationAnalyzer and remove the
original copy inside clangtidy.

Diff Detail

Repository
rC Clang

Event Timeline

shuaiwang created this revision.Sep 11 2018, 2:07 PM
george.karpenkov accepted this revision.Sep 11 2018, 2:08 PM

Wow, thanks a lot, I'm really looking forward to using this!

This revision is now accepted and ready to land.Sep 11 2018, 2:08 PM
JonasToth added a project: Restricted Project.
This revision was automatically updated to reflect the committed changes.
This revision was automatically updated to reflect the committed changes.
thakis added a subscriber: thakis.Jan 4 2023, 5:03 PM

Old review, but clang binary should not depend on ASTMatchers.

It's fine if ExprMutationAnalyzer isn't used within clang, but then it should go into a different library, not into clang/lib/Analysis, which is used by clang.

Herald added a project: Restricted Project. · View Herald TranscriptJan 4 2023, 5:03 PM