[PATCH] An analysis to find external function pointers and trace their data flow

Tom Roeder tmroeder at google.com
Wed Mar 12 12:17:36 PDT 2014


  This diff makes changes as suggested by the last review. In summary:

  - Fixed the minor points noted in JF's comments
  - C++11-ified loops and such. Please let me know if there's other C++11-ification I should do here
  - Switched to using TargetLibraryInfo instead of hardcoded strings to check for memory-allocation functions.
  - Switched to using LLVM ADT as much as possible; I still need std::list in a couple places for the iterator guarantees it gives
  - The switch to C++11 range loops forced a bunch of const throughout the code. This probably should have been there in the first place.
  - Added explanatory docs on the class.

http://llvm-reviews.chandlerc.com/D2873

CHANGE SINCE LAST DIFF
  http://llvm-reviews.chandlerc.com/D2873?vs=7326&id=7781#toc

Files:
  include/llvm/Analysis/ExternalFunctionAnalysis.h
  include/llvm/InitializePasses.h
  lib/Analysis/Analysis.cpp
  lib/Analysis/CMakeLists.txt
  lib/Analysis/ExternalFunctionAnalysis.cpp
  test/Analysis/ExternalFunctionAnalysis/external_function_analysis.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D2873.2.patch
Type: text/x-patch
Size: 35011 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140312/3225be43/attachment.bin>


More information about the llvm-commits mailing list