r212393 - Fix the MicrosoftExtensions.c target

[Aaron Ballman]

On Sun, Jul 6, 2014 at 1:58 AM, Ehsan Akhgari <ehsan.akhgari at gmail.com> wrote:
> Author: ehsan
> Date: Sun Jul  6 00:58:55 2014
> New Revision: 212393
>
> URL: http://llvm.org/viewvc/llvm-project?rev=212393&view=rev
> Log:
> Fix the MicrosoftExtensions.c target
>
> Modified:
>     cfe/trunk/test/Parser/MicrosoftExtensions.c
>
> Modified: cfe/trunk/test/Parser/MicrosoftExtensions.c
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Parser/MicrosoftExtensions.c?rev=212393&r1=212392&r2=212393&view=diff
> ==============================================================================
> --- cfe/trunk/test/Parser/MicrosoftExtensions.c (original)
> +++ cfe/trunk/test/Parser/MicrosoftExtensions.c Sun Jul  6 00:58:55 2014
> @@ -1,3 +1,4 @@
> +// REQUIRES: x86-registered-target
>  // RUN: %clang_cc1 -triple i386-mingw32 -fsyntax-only -verify -fms-extensions  -Wno-missing-declarations -x objective-c++ %s

I don't believe this is the correct fix for this problem. Please split
the inline assembly out into its own test instead
(MicrosoftExtensions-x86.c). That way, the remaining parts of this
test will run on all of the bots. (This is a very general test file
for MS extensions, so reducing test coverage is a bad thing).

Thanks!

~Aaron

>  __stdcall int func0();
>  int __stdcall func();
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits