[PATCH] D21475: [CFLAA] Summarize interprocedural aliasing information instead of recomputing it at callsite

Mandeep Singh Grang via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 17 11:51:58 PDT 2016


mgrang added a subscriber: mgrang.

================
Comment at: lib/Analysis/CFLAliasAnalysis.cpp:71
@@ +70,3 @@
+/// return value. When the index is 0, they represent the return value. Non-zero
+/// index i represents the i-th parameter
+struct ExternalRelation {
----------------
Period needed at end of comment.

================
Comment at: lib/Analysis/CFLAliasAnalysis.cpp:80
@@ -72,3 +79,3 @@
 
-  FunctionInfo(StratifiedSets<Value *> &&S, SmallVector<Value *, 4> &&RV)
-      : Sets(std::move(S)), ReturnedValues(std::move(RV)) {}
+  // RetParamRelations is a collection of ExternalRelations
+  SmallVector<ExternalRelation, 8> RetParamRelations;
----------------
Period needed at end of comment.

================
Comment at: lib/Analysis/CFLAliasAnalysis.cpp:111
@@ -103,1 +110,3 @@
 
+/// The maximum number of arguments we can put into a summary
+LLVM_CONSTEXPR unsigned MaxSupportedArgsInSummary = 50;
----------------
Period needed at end of comment.


http://reviews.llvm.org/D21475





More information about the llvm-commits mailing list