<div dir="ltr">Sounds great. Thanks!<div><br></div><div>-eric<br><br><div class="gmail_quote"><div dir="ltr">On Wed, May 4, 2016 at 4:36 AM Andrey Turetskiy <<a href="mailto:andrey.turetskiy@gmail.com">andrey.turetskiy@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">aturetsk added a comment.<br>
<br>
Hi,<br>
Thanks for the review.<br>
<br>
Committed:<br>
<br>
1. Add a test for driver options from m_x86_Features_Group (<a href="http://reviews.llvm.org/rL268487" rel="noreferrer" target="_blank">http://reviews.llvm.org/rL268487</a>)<br>
2. Add missing -mno-cx16 driver option (<a href="http://reviews.llvm.org/rL268488" rel="noreferrer" target="_blank">http://reviews.llvm.org/rL268488</a>)<br>
3. Add -m[no-]x87 and -m[no-]80387 options to control FeatureX87 (<a href="http://reviews.llvm.org/rL268489" rel="noreferrer" target="_blank">http://reviews.llvm.org/rL268489</a>)<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D19658" rel="noreferrer" target="_blank">http://reviews.llvm.org/D19658</a><br>
<br>
<br>
<br>
</blockquote></div></div></div>