<div dir="ltr">I reverted it because it broke tests, waiting on Stephen to finish up the fixed patch you link to. I'm not in any rush to get it into 3.6, so I'd just wait until the next release instead of merging.</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 15, 2015 at 10:08 AM, Hans Wennborg <span dir="ltr"><<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Jan 13, 2015 at 5:07 PM, JF Bastien <<a href="mailto:jfb@google.com">jfb@google.com</a>> wrote:<br>
> Author: jfb<br>
> Date: Tue Jan 13 19:07:51 2015<br>
> New Revision: 225910<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=225910&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=225910&view=rev</a><br>
> Log:<br>
> Insert random noops to increase security against ROP attacks (clang)<br>
><br>
> A pass that adds random noops to X86 binaries to introduce diversity with the goal of increasing security against most return-oriented programming attacks.<br>
><br>
> Command line options:<br>
>   -noop-insertion // Enable noop insertion.<br>
>   -noop-insertion-percentage=X // X% of assembly instructions will have a noop prepended (default: 50%, requires -noop-insertion)<br>
>   -max-noops-per-instruction=X // Randomly generate X noops per instruction. ie. roll the dice X times with probability set above (default: 1). This doesn't guarantee X noop instructions.<br>
><br>
> In addition, the following 'quick switch' in clang enables basic diversity using default settings (currently: noop insertion and schedule randomization; it is intended to be extended in the future).<br>
>   -fdiversify<br>
><br>
> This is the clang part of the patch.<br>
> llvm part: D3392<br>
><br>
> <a href="http://reviews.llvm.org/D3393" target="_blank">http://reviews.llvm.org/D3393</a><br>
> Patch by Stephen Crane (@rinon)<br>
<br>
</span>Is this something that should be mentioned in the 3.6 release notes?<br>
<br>
Also are there patches in flight here that need to get merged? (I saw<br>
<a href="http://reviews.llvm.org/D6983" target="_blank">http://reviews.llvm.org/D6983</a> for example)<br>
<br>
Thanks,<br>
Hans<br>
</blockquote></div><br></div>