[llvm-commits] [test-suite] r171017 - /test-suite/trunk/SingleSource/UnitTests/Vectorizer/gcc-loops.cpp

[David Blaikie]

On Sun, Dec 23, 2012 at 10:05 PM, Nadav Rotem <nrotem at apple.com> wrote:
> Author: nadav
> Date: Mon Dec 24 00:05:58 2012
> New Revision: 171017
>
> URL: http://llvm.org/viewvc/llvm-project?rev=171017&view=rev
> Log:
> Fix warnings
>
> Modified:
>     test-suite/trunk/SingleSource/UnitTests/Vectorizer/gcc-loops.cpp

Any reason we're not just turning off warnings when building  these tests?

>
> Modified: test-suite/trunk/SingleSource/UnitTests/Vectorizer/gcc-loops.cpp
> URL: http://llvm.org/viewvc/llvm-project/test-suite/trunk/SingleSource/UnitTests/Vectorizer/gcc-loops.cpp?rev=171017&r1=171016&r2=171017&view=diff
> ==============================================================================
> --- test-suite/trunk/SingleSource/UnitTests/Vectorizer/gcc-loops.cpp (original)
> +++ test-suite/trunk/SingleSource/UnitTests/Vectorizer/gcc-loops.cpp Mon Dec 24 00:05:58 2012
> @@ -78,8 +78,6 @@
>
>  __attribute__((noinline))
>  void example4a (int n, aint * __restrict__ p, aint * __restrict__ q) {
> -   int i;
> -    const int MAX = 4;
>     /* feature: support for (aligned) pointer accesses  */
>     /* feature: support for constants  */
>     while (n--){
> @@ -385,7 +383,7 @@
>
>    std::cout<<std::hex;
>    std::cout<<"Results: ("<<std::accumulate(results.begin(), results.end(), 0)<<"):";
> -  for (int i=0; i < results.size(); ++i) {
> +  for (unsigned i=0; i < results.size(); ++i) {
>      std::cout<<" "<<results[i];
>    }
>    std::cout<<"\n";
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits