Uh, we should always be evaluating the call even if we can't inline it. In fact, if we //are// evaluating it, we can probably drop the part of VisitCXXNewExpr that calls invalidateRegions for the allocator, even when inlining is off. http://llvm-reviews.chandlerc.com/D2646