<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sun, Jul 6, 2014 at 10:03 AM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span> wrote:<br>

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


> Author: ehsan<br>
> Date: Sun Jul  6 00:58:55 2014<br>
> New Revision: 212393<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=212393&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=212393&view=rev</a><br>
> Log:<br>
> Fix the MicrosoftExtensions.c target<br>
><br>
> Modified:<br>
>     cfe/trunk/test/Parser/MicrosoftExtensions.c<br>
><br>
> Modified: cfe/trunk/test/Parser/MicrosoftExtensions.c<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Parser/MicrosoftExtensions.c?rev=212393&r1=212392&r2=212393&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Parser/MicrosoftExtensions.c?rev=212393&r1=212392&r2=212393&view=diff</a><br>


> ==============================================================================<br>
> --- cfe/trunk/test/Parser/MicrosoftExtensions.c (original)<br>
> +++ cfe/trunk/test/Parser/MicrosoftExtensions.c Sun Jul  6 00:58:55 2014<br>
> @@ -1,3 +1,4 @@<br>
> +// REQUIRES: x86-registered-target<br>
>  // RUN: %clang_cc1 -triple i386-mingw32 -fsyntax-only -verify -fms-extensions  -Wno-missing-declarations -x objective-c++ %s<br>
<br>
I don't believe this is the correct fix for this problem. Please split<br>
the inline assembly out into its own test instead<br>
(MicrosoftExtensions-x86.c). That way, the remaining parts of this<br>
test will run on all of the bots. (This is a very general test file<br>
for MS extensions, so reducing test coverage is a bad thing).<br></blockquote><div><br>Done: <a href="http://reviews.llvm.org/D4401">http://reviews.llvm.org/D4401</a><br><br>Sorry about this!<br><br clear="all"><div>--<br>

Ehsan<br><<a href="http://ehsanakhgari.org/">http://ehsanakhgari.org/</a>></div></div></div></div></div>