[analyzer] Begin replacing ObjCMessage with ObjCMethodCall and friends.

Previously, the CallEvent subclass ObjCMessageInvocation was just a wrapper
around the existing ObjCMessage abstraction (over message sends and property
accesses). Now, we have abstract CallEvent ObjCMethodCall with subclasses
ObjCMessageSend and ObjCPropertyAccess.

In addition to removing yet another wrapper object, this should make it easy
to add a ObjCSubscriptAccess call event soon.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@159558 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/lib/StaticAnalyzer/Checkers/ObjCSelfInitChecker.cpp b/lib/StaticAnalyzer/Checkers/ObjCSelfInitChecker.cpp
index bb6ab6f..c25da87 100644
--- a/lib/StaticAnalyzer/Checkers/ObjCSelfInitChecker.cpp
+++ b/lib/StaticAnalyzer/Checkers/ObjCSelfInitChecker.cpp
@@ -209,7 +209,9 @@
     return;
   }
 
-  ObjCMessageInvocation MsgWrapper(msg, C.getState(), C.getLocationContext());
+  // FIXME: ObjCMessage is going away.
+  ObjCMessageSend MsgWrapper(msg.getMessageExpr(), C.getState(),
+                             C.getLocationContext());
   checkPostStmt(MsgWrapper, C);
 
   // We don't check for an invalid 'self' in an obj-c message expression to cut
@@ -300,7 +302,9 @@
 
 void ObjCSelfInitChecker::checkPreObjCMessage(ObjCMessage Msg,
                                               CheckerContext &C) const {
-  ObjCMessageInvocation MsgWrapper(Msg, C.getState(), C.getLocationContext());
+  // FIXME: ObjCMessage is going away.
+  ObjCMessageSend MsgWrapper(Msg.getMessageExpr(), C.getState(),
+                             C.getLocationContext());
   checkPreStmt(MsgWrapper, C);
 }