Initialize BasicAAWrapperPass in it's constructor

[loladiro]

Summary

This seems to be the idiom used elsewhere in LLVM. Is there some reason this shouldn't be there or is that just an oversight?

[loladiro]

bump

[chandlerc]

Probably just an oversight. Thanks for the fix, LGTM.

include/llvm/Analysis/BasicAliasAnalysis.h

 public:
   static char ID;
 
-  BasicAAWrapperPass() : FunctionPass(ID) {}
+  BasicAAWrapperPass();
 
   BasicAAResult &getResult() { return *Result; }
   const BasicAAResult &getResult() const { return *Result; }

lib/Analysis/BasicAliasAnalysis.cpp

                        AM->getCachedResult<LoopAnalysis>(F));
 }
 
+BasicAAWrapperPass::BasicAAWrapperPass() : FunctionPass(ID) {
+    initializeBasicAAWrapperPassPass(*PassRegistry::getPassRegistry());
+}
+
 char BasicAAWrapperPass::ID = 0;
 void BasicAAWrapperPass::anchor() {}