r213170 - clang-cl: expand test coverage for "core" options

Hans Wennborg hans at chromium.org
Wed Jul 16 11:29:48 PDT 2014


Thanks for catching that! I've removed that line in r213179 since it's
covered by cl-x86-flags.c (which we shuold also fix) anyway.

On Wed, Jul 16, 2014 at 11:10 AM, Reid Kleckner <rnk at google.com> wrote:
> The -msse2 test won't work if clang-cl isn't targeting x86.  See the
> cl-x86-flags.c test which is failing here:
> http://lab.llvm.org:8011/builders/clang-ppc64-elf-linux2/builds/15629
>
>
> On Wed, Jul 16, 2014 at 9:29 AM, Hans Wennborg <hans at hanshq.net> wrote:
>>
>> Author: hans
>> Date: Wed Jul 16 11:29:00 2014
>> New Revision: 213170
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=213170&view=rev
>> Log:
>> clang-cl: expand test coverage for "core" options
>>
>> Modified:
>>     cfe/trunk/test/Driver/cl-options.c
>>
>> Modified: cfe/trunk/test/Driver/cl-options.c
>> URL:
>> http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/cl-options.c?rev=213170&r1=213169&r2=213170&view=diff
>>
>> ==============================================================================
>> --- cfe/trunk/test/Driver/cl-options.c (original)
>> +++ cfe/trunk/test/Driver/cl-options.c Wed Jul 16 11:29:00 2014
>> @@ -294,9 +294,16 @@
>>
>>  // Accept "core" clang options.
>>  // (/Zs is for syntax-only)
>> -// RUN: %clang_cl /Zs \
>> +// RUN: %clang_cl \
>> +// RUN:     --driver-mode=cl \
>> +// RUN:     -ferror-limit=10 \
>>  // RUN:     -fmsc-version=1800 \
>> -// RUN:     -- %s 2>&1
>> +// RUN:     -fno-strict-aliasing \
>> +// RUN:     -fstrict-aliasing \
>> +// RUN:     -mllvm -disable-llvm-optzns \
>> +// RUN:     -msse2 \
>> +// RUN:     -Wunused-variables \
>> +// RUN:     /Zs -- %s 2>&1
>>
>>
>>  void f() { }
>>
>>
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>



More information about the cfe-commits mailing list