[PATCH] D15611: [Patch 2/3]: Rebasing Ryan Govostes' STP patch for Clang SA

Bhargava Shastry via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 7 03:07:56 PST 2016


bshastry added a comment.

In http://reviews.llvm.org/D15611#320699, @zaks.anna wrote:

> What was the last outstanding issue with this patch (I believe it was related to either make or cmake changes)? 
>  Has it been addressed?


I have tested on a Linux machine and can confirm that it works as documented in the README. The cmake bit of the patch is actually in the third and final sequence of patches for this issue [1].

Caveats:

- stp APIs that the plugin uses need to be statically linked
  - I have only known stp being used by an executable (KLEE) where dynamic linking is a possibility
  - For something that is itself a shared object library (this plugin), static linking is the work-around that I chose

[1]: http://reviews.llvm.org/D15612


Repository:
  rL LLVM

http://reviews.llvm.org/D15611





More information about the cfe-commits mailing list