[PATCH] D13628: Initialize BasicAAWrapperPass in it's constructor

[Keno Fischer via llvm-commits]

loladiro created this revision.
loladiro added a reviewer: chandlerc.
loladiro added a subscriber: llvm-commits.
loladiro set the repository for this revision to rL LLVM.

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?

Repository:
  rL LLVM

http://reviews.llvm.org/D13628

Files:
  include/llvm/Analysis/BasicAliasAnalysis.h
  lib/Analysis/BasicAliasAnalysis.cpp

Index: lib/Analysis/BasicAliasAnalysis.cpp
===================================================================
--- lib/Analysis/BasicAliasAnalysis.cpp
+++ lib/Analysis/BasicAliasAnalysis.cpp
@@ -1489,6 +1489,10 @@
                        AM->getCachedResult<LoopAnalysis>(F));
 }
 
+BasicAAWrapperPass::BasicAAWrapperPass() : FunctionPass(ID) {
+    initializeBasicAAWrapperPassPass(*PassRegistry::getPassRegistry());
+}
+
 char BasicAAWrapperPass::ID = 0;
 void BasicAAWrapperPass::anchor() {}
 
Index: include/llvm/Analysis/BasicAliasAnalysis.h
===================================================================
--- include/llvm/Analysis/BasicAliasAnalysis.h
+++ include/llvm/Analysis/BasicAliasAnalysis.h
@@ -243,7 +243,7 @@
 public:
   static char ID;
 
-  BasicAAWrapperPass() : FunctionPass(ID) {}
+  BasicAAWrapperPass();
 
   BasicAAResult &getResult() { return *Result; }
   const BasicAAResult &getResult() const { return *Result; }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D13628.37044.patch
Type: text/x-patch
Size: 944 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151011/e43aa7ed/attachment.bin>