Looks good after comment nit fix. ================ Comment at: include/clang/ASTMatchers/ASTMatchers.h:551 @@ +550,3 @@ +/// +/// Example matches "this" before i. +/// (matcher = thisExpr()) ---------------- Perhaps rephrase to: Example matches the implicit this expression in "return i". http://llvm-reviews.chandlerc.com/D70