[PATCH] Insert random NOPs to increase security against ROP attacks (clang)

[Stephen Crane]

Ping. Anyone more familiar with clang who can take a quick look at
this patch? Just a couple flags getting passed through to the backend.

Thanks,
Stephen

On Fri, Dec 19, 2014 at 3:32 PM, JF Bastien <jfb at chromium.org> wrote:
> Can you update the description to use "noop" instead of "nop"? I don't really care either way, but it should be consistent with the code.
>
> It also looks like this patch only passes `-noop-insertion` to LLVM, not the other 2?
>
> I recommend seeking an LGTM from a person more familiar with clang that I am. This looks OK to my untrained eye, but I'd not at all familiar with clang's code base.
>
>
> http://reviews.llvm.org/D3393
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/
>
>