[PATCH] D34638: [CFLAA] Change FunctionHandle to be common to Steensgaard's and Andersens'

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 26 12:18:15 PDT 2017


davide created this revision.
Herald added a subscriber: david2050.

In order to fix https://bugs.llvm.org/show_bug.cgi?id=33339 we need to keep CallBackVH for instructions inserted in the stratified sets so that we can remove when, e.g. instcombine or any other transform remove the instruction.

When looking at fixing this I noticed that Andersens' and Steensgaard's analyses end up duplicating a considerable amount of code, including the cache. This is a first step towards the unification of what's common to the two analyses.


https://reviews.llvm.org/D34638

Files:
  include/llvm/Analysis/CFLAliasAnalysisUtils.h
  include/llvm/Analysis/CFLAndersAliasAnalysis.h
  include/llvm/Analysis/CFLSteensAliasAnalysis.h
  lib/Analysis/CFLAndersAliasAnalysis.cpp
  lib/Analysis/CFLSteensAliasAnalysis.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D34638.104005.patch
Type: text/x-patch
Size: 6532 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170626/f4061a00/attachment.bin>


More information about the llvm-commits mailing list