flx added a comment.
This looks great, but goes past my knowledge of the AST API :) While the
existing and new test coverage provides good confidence, one other reviewer
taking look would be good.
================
Comment at: clang-tools-extra/clang-tidy/utils/DeclRefExprUtils.cpp:112
+ switch (
+ E->ClassifyModifiable(Finder->getASTContext(), Loc).getModifiable())
{
+ case Expr::Classification::CM_Untested:
----------------
Reuse Ctx here?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D114539/new/
https://reviews.llvm.org/D114539
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
