This last patch is definitely fine to at least unbreak these tests on such machines. I'll try to commit it tomorrow if someone else doesn't beat me to it.<div><br></div><div>I'm actually wonderig if we should go with something more akin to your first though, and start being more explicit in which feature sets are under test where... Anyone else on the list have thoughts here?</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jul 12, 2012 at 2:22 AM, Michael Liao <span dir="ltr"><<a href="mailto:michael.liao@intel.com" target="_blank">michael.liao@intel.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The patch is simplified to only disable AVX2 feature for that tests by<br>
adding '-mcpu=corei7'.<br>
<br>
Yours<br>
- Michael<br>
<div><div class="h5"><br>
On Wed, 2012-07-11 at 09:59 -0700, Michael Liao wrote:<br>
> Hi,<br>
><br>
> Please review the patch on revising some test cases without tight test<br>
> parameters. On machine with AVX2/BMI features, these tests will be with<br>
> AVX2/BMI enabled and fails due to different code generation.<br>
><br>
> Yours<br>
> - Michael<br>
</div></div>> _______________________________________________<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>_______________________________________________<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></blockquote></div><br></div>