[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