<div dir="ltr">Committed in r218560.</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 26, 2014 at 11:22 AM, Steve King <span dir="ltr"><<a href="mailto:steve@metrokings.com" target="_blank">steve@metrokings.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks Tom and Craig.<br>
<br>
I do not have commit access.  Can someone please commit patch 14123?<br>
<div class="HOEnZb"><div class="h5"><br>
On Fri, Sep 26, 2014 at 10:29 AM, Tom Stellard <<a href="mailto:tom@stellard.net">tom@stellard.net</a>> wrote:<br>
> On Fri, Sep 26, 2014 at 09:22:36AM -0700, Steve King wrote:<br>
>> On Fri, Sep 26, 2014 at 6:44 AM, Tom Stellard <<a href="mailto:tom@stellard.net">tom@stellard.net</a>> wrote:<br>
>> ><br>
>> > Tablegen has !shl and !srl operators, I think it would be good to add test<br>
>> > cases using those too.<br>
>> ><br>
>><br>
>> Hi Tom - How would you use the ! operators here?  The test verifies<br>
>> general cherry picking of bits from operand to instruction.  Shift<br>
>> wasn't the right term -- I should rename the test as<br>
>> "BitOffsetDecoder".<br>
><br>
> Hi Steve,<br>
><br>
> Sorry, I didn't look quite close enough at what you were doing.<br>
><br>
> In my head I was thinking it was something like:<br>
><br>
> def Inst {<br>
><br>
>   field bits<5> src;<br>
><br>
>   let Inst{2-0} = !srl(src, 2);<br>
> }<br>
><br>
> I think the patch is OK as is.<br>
><br>
> -Tom<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>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>~Craig
</div>