[PATCH] D25455: Document potential implementation of CFI in hardware.

Kostya Serebryany via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 10 16:50:18 PDT 2016


kcc added a comment.

In https://reviews.llvm.org/D25455#566834, @pcc wrote:

> I think this would benefit from an assembly code listing for a couple of cases (e.g. all-ones, bit vector).
>
> One concern I have is that in the case where we have a bit vector lookup, the code would end up computing the byte offset redundantly with the instruction, which would somewhat diminish the code size benefits here.


Right now I don't see a good way to have the bit vector lookup in the HW scheme. (written: "The bit vector lookup is probably too complex for a hardware implementation.")
Even w/o the bit vector the check will nearly precise (and fully precise in > 50% cases).


https://reviews.llvm.org/D25455





More information about the llvm-commits mailing list