[PATCH] D22009: [CFLAA] Simplify CFLGraphBuilder by removing the escaped value set and external value set

Jia Chen via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 5 13:11:18 PDT 2016


grievejia created this revision.
grievejia added reviewers: george.burgess.iv, hfinkel.
grievejia added a subscriber: llvm-commits.

This is a code cleanup patch. No functionality change is introduced.

Previously, CFLGraphBuilder maintained 5 auxiliary structures in addition to the CFLGraph under construction. Among those five structures, the escaped value set and the external value set were irreplaceable when they were first introduced, but now their use cases can be completely covered by InstantiatedAttrs. We could simplify CFLGraphBuilder (as well as its clients) by removing those two sets.

http://reviews.llvm.org/D22009

Files:
  lib/Analysis/CFLSteensAliasAnalysis.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D22009.62779.patch
Type: text/x-patch
Size: 5760 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160705/41931e2f/attachment.bin>


More information about the llvm-commits mailing list