[PATCH] New alias analysis for static global variables

hfinkel at anl.gov hfinkel at anl.gov
Mon Jun 1 05:42:49 PDT 2015


It looks like you just uploaded a diff against the previous patch. Please upload the full diff against trunk instead (with full context).


================
Comment at: lib/Transforms/IPO/PassManagerBuilder.cpp:202
@@ -201,2 +201,3 @@
   addInitialAliasAnalysisPasses(MPM);
+  MPM.add(createStaticGlobalAAPass());
 
----------------
samparker wrote:
> hfinkel wrote:
> > Should this be part of the preceeding function?
> I tried that, but it caused LLVM to crash. I believe its because that function is also called by populateFunctionPassManager function.
Okay; please add a comment noting that.

http://reviews.llvm.org/D10059

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list