<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jul 17, 2015 at 5:52 AM, Alexey Frolov <span dir="ltr"><<a href="mailto:alexfrolov1878@yandex.ru" target="_blank">alexfrolov1878@yandex.ru</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">alexfrol added a comment.<br>
<span class=""><br>
In <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D10536-23203421&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=BSqEv9KvKMW_Ob8SyngJ70KdZISM_ASROnREeq0cCxk&m=u3CcqjIoERsKpg0HB4Ds_N01tFBBS1kr4WNYmjXPUvk&s=Fin1Bmpfa9qoNNjTDZQUpRI1pFH30bRkFnn4Kr5kwEo&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/D10536#203421</a>, @compnerd wrote:<br>
<br>
> LGTM with the small change to rename isValidAsmValue at least.<br>
<br>
<br>
</span>All issues are fixed. I'll update the patch soon.<br>
<span class=""><br>
In <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D10536-23203820&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=BSqEv9KvKMW_Ob8SyngJ70KdZISM_ASROnREeq0cCxk&m=u3CcqjIoERsKpg0HB4Ds_N01tFBBS1kr4WNYmjXPUvk&s=KlRfntEilRh3rGMws6lc5k_i05o_75i_iOTAfDDk8OY&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/D10536#203820</a>, @echristo wrote:<br>
<br>
> This will need documentation updates yes? Also backend support?<br>
><br>
> -eric<br>
<br>
<br>
</span>What documentation updates do you mean?<br></blockquote><div><br></div><div>Additions/modifications to:</div><div><a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_docs_LangRef.html-23inline-2Dassembler-2Dexpressions&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=BSqEv9KvKMW_Ob8SyngJ70KdZISM_ASROnREeq0cCxk&m=u3CcqjIoERsKpg0HB4Ds_N01tFBBS1kr4WNYmjXPUvk&s=flz5OqWJ4huQR1l9sE-2f2WvTSOfXZGErmrwEH9PT5E&e=">http://llvm.org/docs/LangRef.html#inline-assembler-expressions</a></div><div><br></div><div>-- Sean Silva</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
Backend support is necessary for ‘v’, ‘k’, and ‘Yk’ constraints. They are AVX-512 specific and as far as I know work for AVX512 backend support is still ongoing.<br>
<span class=""><br>
In <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D10536-23203821&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=BSqEv9KvKMW_Ob8SyngJ70KdZISM_ASROnREeq0cCxk&m=u3CcqjIoERsKpg0HB4Ds_N01tFBBS1kr4WNYmjXPUvk&s=qRI3xU9f9usmHdrsrreF-kyBo6aAfwowB9gvGcbmOHA&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/D10536#203821</a>, @echristo wrote:<br>
<br>
> I don't see a test for the 'k' support or a few other things either. Can you split this out a bit into the various parts for checking the existing constraints, and adding individual ones?<br>
><br>
> -eric<br>
<br>
<br>
</span>Sure, will do.<br>
<div class=""><div class="h5"><br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D10536&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=BSqEv9KvKMW_Ob8SyngJ70KdZISM_ASROnREeq0cCxk&m=u3CcqjIoERsKpg0HB4Ds_N01tFBBS1kr4WNYmjXPUvk&s=YPdpK7GK2E3CdQDqS5cP_Fm1rMQ_-a6GwIDayeFDZD0&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/D10536</a><br>
<br>
<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div></div>