<div dir="ltr">On Tue, Jul 15, 2014 at 12:01 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="">On Mon, Jul 14, 2014 at 7:41 PM, Ehsan Akhgari <<a href="mailto:ehsan.akhgari@gmail.com">ehsan.akhgari@gmail.com</a>> wrote:<br>


> On Mon, Jul 14, 2014 at 10:34 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
>><br>
>> On Mon, Jul 14, 2014 at 1:28 PM, Ehsan Akhgari <<a href="mailto:ehsan.akhgari@gmail.com">ehsan.akhgari@gmail.com</a>><br>
>> wrote:<br>
>> > Author: ehsan<br>
>> > Date: Mon Jul 14 15:28:21 2014<br>
>> > New Revision: 212979<br>
>> ><br>
>> > URL: <a href="http://llvm.org/viewvc/llvm-project?rev=212979&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=212979&view=rev</a><br>
>> > Log:<br>
>> > Add support for intercepting thunks of the form<br>
>><br>
>> Is this a complete commit message (or a complete commit? Missing tests?)?<br>
><br>
><br>
> No, sorry, this commit used to have "`jmp dword ptr ds:[WWZZYYXX]`" after<br>
> it, but either git or git svn ate that part, and I didn't realize.  The code<br>
> changes are complete.<br>
<br>
</div>Is there test coverage for this? Any way to add it?<br><div class=""><div class="h5"></div></div></blockquote><div><br></div><div>No, I don't know of a way to test this, and I couldn't find any existing tests for the rest of the cases.  Timur, do you know if there is a way to test this?<br>

</div><div><br clear="all"><div>--<br>Ehsan<br><<a href="http://ehsanakhgari.org/">http://ehsanakhgari.org/</a>></div><br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

<div class=""><div class="h5">
><br>
> Cheers,<br>
> --<br>
> Ehsan<br>
> <<a href="http://ehsanakhgari.org/" target="_blank">http://ehsanakhgari.org/</a>><br>
><br>
>><br>
>> ><br>
>> > Modified:<br>
>> >     compiler-rt/trunk/lib/interception/interception_win.cc<br>
>> ><br>
>> > Modified: compiler-rt/trunk/lib/interception/interception_win.cc<br>
>> > URL:<br>
>> > <a href="http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/interception/interception_win.cc?rev=212979&r1=212978&r2=212979&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/interception/interception_win.cc?rev=212979&r1=212978&r2=212979&view=diff</a><br>


>> ><br>
>> > ==============================================================================<br>
>> > --- compiler-rt/trunk/lib/interception/interception_win.cc (original)<br>
>> > +++ compiler-rt/trunk/lib/interception/interception_win.cc Mon Jul 14<br>
>> > 15:28:21 2014<br>
>> > @@ -113,6 +113,7 @@ static size_t RoundUpToInstrBoundary(siz<br>
>> >          cursor += 3;<br>
>> >          continue;<br>
>> >        case 0xC1F7:  // F7 C1 XX YY ZZ WW = test ecx, WWZZYYXX<br>
>> > +      case 0x25FF:  // FF 25 XX YY ZZ WW = jmp dword ptr ds:[WWZZYYXX]<br>
>> >          cursor += 6;<br>
>> >          continue;<br>
>> >        case 0x3D83:  // 83 3D XX YY ZZ WW TT = cmp TT, WWZZYYXX<br>
>> ><br>
>> ><br>
>> > _______________________________________________<br>
>> > llvm-commits mailing list<br>
>> > <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
>> > <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
><br>
><br>
</div></div></blockquote></div><br></div></div>