[PATCH] Add Forward-Edge Control-Flow Integrity support

Kostya Serebryany kcc at google.com
Fri Jul 11 02:20:08 PDT 2014


================
Comment at: include/llvm/CodeGen/ForwardControlFlowIntegrity.h:56
@@ +55,3 @@
+private:
+  typedef DenseSet<Instruction *> CallSet;
+
----------------
Tom Roeder wrote:
> Kostya Serebryany wrote:
> > Can this be SmallVector?
> I don't know SmallVector well, so I'm not sure. It's true that I don't need efficient search over this data structure, since it's just filled and iterated, so at least it could be a vector. Does SmallVector suffer if it gets large?
SmallVector becomes just std::vector when it becomes large.

http://reviews.llvm.org/D4167






More information about the llvm-commits mailing list