CSA emits results for kiss_fft

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

CSA emits results for kiss_fft

August Joki
The CSA included with the latest release of Xcode (3.2.3) emits 21
analyzer results for the latest (v1.2.9) version of kiss_fft. It looks
like it's having issues with the macros. Is this a known issue?

-August

kiss_fft:
http://sourceforge.net/projects/kissfft/files/kissfft/v1_2_9/

Analyzer Results:
kiss_fft_v1_2_9/kiss_fft.c:40:11:{40:9-40:15}: warning: Value stored
to 'Fout' is never read
         ++Fout;
         ~~^~~~
kiss_fft_v1_2_9/kiss_fft.c:39:11:{39:9-39:16}: warning: Value stored
to 'Fout2' is never read
         ++Fout2;
         ~~^~~~~
kiss_fft_v1_2_9/kiss_fft.c:36:9:{36:16-36:23}: warning: Value stored
to 'tw1' is never read
         tw1 += fstride;
         ^      ~~~~~~~
kiss_fft_v1_2_9/kiss_fft.c:31:5:{31:13-31:21}: warning: Value stored
to 'Fout2' is never read
     Fout2 = Fout + m;
     ^       ~~~~~~~~
kiss_fft_v1_2_9/kiss_fft.c:29:20:{29:26-29:38}: warning: Value stored
to 'tw1' during its initialization is never read
     kiss_fft_cpx * tw1 = st->twiddles;
                    ^     ~~~~~~~~~~~~
kiss_fft_v1_2_9/kiss_fft.c:88:11:{88:9-88:15}: warning: Value stored
to 'Fout' is never read
         ++Fout;
         ~~^~~~
kiss_fft_v1_2_9/kiss_fft.c:74:9:{74:16-74:25}: warning: Value stored
to 'tw3' is never read
         tw3 += fstride*3;
         ^      ~~~~~~~~~
kiss_fft_v1_2_9/kiss_fft.c:73:9:{73:16-73:25}: warning: Value stored
to 'tw2' is never read
         tw2 += fstride*2;
         ^      ~~~~~~~~~
kiss_fft_v1_2_9/kiss_fft.c:72:9:{72:16-72:23}: warning: Value stored
to 'tw1' is never read
         tw1 += fstride;
         ^      ~~~~~~~
kiss_fft_v1_2_9/kiss_fft.c:58:11:{58:17-58:35}: warning: Although the
value stored to 'tw2' is used in the enclosing expression, the value
is never actually read from 'tw2'
     tw3 = tw2 = tw1 = st->twiddles;
           ^     ~~~~~~~~~~~~~~~~~~
kiss_fft_v1_2_9/kiss_fft.c:58:5:{58:11-58:35}: warning: Value stored
to 'tw3' is never read
     tw3 = tw2 = tw1 = st->twiddles;
     ^     ~~~~~~~~~~~~~~~~~~~~~~~~
kiss_fft_v1_2_9/kiss_fft.c:58:17:{58:23-58:35}: warning: Although the
value stored to 'tw1' is used in the enclosing expression, the value
is never actually read from 'tw1'
     tw3 = tw2 = tw1 = st->twiddles;
                 ^     ~~~~~~~~~~~~
kiss_fft_v1_2_9/kiss_fft.c:55:18:{55:21-55:24}: warning: Value stored
to 'm3' during its initialization is never read
     const size_t m3=3*m;
                  ^  ~~~
kiss_fft_v1_2_9/kiss_fft.c:54:18:{54:21-54:24}: warning: Value stored
to 'm2' during its initialization is never read
     const size_t m2=2*m;
                  ^  ~~~
kiss_fft_v1_2_9/kiss_fft.c:117:10:{117:17-117:26}: warning: Value
stored to 'tw2' is never read
          tw2 += fstride*2;
          ^      ~~~~~~~~~
kiss_fft_v1_2_9/kiss_fft.c:132:12:{132:10-132:16}: warning: Value
stored to 'Fout' is never read
          ++Fout;
          ~~^~~~
kiss_fft_v1_2_9/kiss_fft.c:116:10:{116:17-116:24}: warning: Value
stored to 'tw1' is never read
          tw1 += fstride;
          ^      ~~~~~~~
kiss_fft_v1_2_9/kiss_fft.c:106:10:{106:14-106:26}: warning: Although
the value stored to 'tw2' is used in the enclosing expression, the
value is never actually read from 'tw2'
      tw1=tw2=st->twiddles;
          ^   ~~~~~~~~~~~~
kiss_fft_v1_2_9/kiss_fft.c:106:6:{106:10-106:26}: warning: Value
stored to 'tw1' is never read
      tw1=tw2=st->twiddles;
      ^   ~~~~~~~~~~~~~~~~
kiss_fft_v1_2_9/kiss_fft.c:104:6:{104:13-104:36}: warning: Value
stored to 'epi3' is never read
      epi3 = st->twiddles[fstride*m];
      ^      ~~~~~~~~~~~~~~~~~~~~~~~
kiss_fft_v1_2_9/kiss_fft.c:100:19:{100:24-100:27}: warning: Value
stored to 'm2' during its initialization is never read
      const size_t m2 = 2*m;
                   ^    ~~~
21 warnings generated.
_______________________________________________
cfe-dev mailing list
[hidden email]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev
Reply | Threaded
Open this post in threaded view
|

Re: CSA emits results for kiss_fft

Ted Kremenek
Hi August,

The CFG looks really broken for the function 'kf_bfly2'.  Please file a bugzilla report so we can track the resolution of this issue.

Thanks,
Ted

On Aug 10, 2010, at 8:00 PM, August Joki wrote:

> The CSA included with the latest release of Xcode (3.2.3) emits 21
> analyzer results for the latest (v1.2.9) version of kiss_fft. It looks
> like it's having issues with the macros. Is this a known issue?
>
> -August
>
> kiss_fft:
> http://sourceforge.net/projects/kissfft/files/kissfft/v1_2_9/
>
> Analyzer Results:
> kiss_fft_v1_2_9/kiss_fft.c:40:11:{40:9-40:15}: warning: Value stored
> to 'Fout' is never read
>         ++Fout;
>         ~~^~~~
> kiss_fft_v1_2_9/kiss_fft.c:39:11:{39:9-39:16}: warning: Value stored
> to 'Fout2' is never read
>         ++Fout2;
>         ~~^~~~~
> kiss_fft_v1_2_9/kiss_fft.c:36:9:{36:16-36:23}: warning: Value stored
> to 'tw1' is never read
>         tw1 += fstride;
>         ^      ~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:31:5:{31:13-31:21}: warning: Value stored
> to 'Fout2' is never read
>     Fout2 = Fout + m;
>     ^       ~~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:29:20:{29:26-29:38}: warning: Value stored
> to 'tw1' during its initialization is never read
>     kiss_fft_cpx * tw1 = st->twiddles;
>                    ^     ~~~~~~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:88:11:{88:9-88:15}: warning: Value stored
> to 'Fout' is never read
>         ++Fout;
>         ~~^~~~
> kiss_fft_v1_2_9/kiss_fft.c:74:9:{74:16-74:25}: warning: Value stored
> to 'tw3' is never read
>         tw3 += fstride*3;
>         ^      ~~~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:73:9:{73:16-73:25}: warning: Value stored
> to 'tw2' is never read
>         tw2 += fstride*2;
>         ^      ~~~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:72:9:{72:16-72:23}: warning: Value stored
> to 'tw1' is never read
>         tw1 += fstride;
>         ^      ~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:58:11:{58:17-58:35}: warning: Although the
> value stored to 'tw2' is used in the enclosing expression, the value
> is never actually read from 'tw2'
>     tw3 = tw2 = tw1 = st->twiddles;
>           ^     ~~~~~~~~~~~~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:58:5:{58:11-58:35}: warning: Value stored
> to 'tw3' is never read
>     tw3 = tw2 = tw1 = st->twiddles;
>     ^     ~~~~~~~~~~~~~~~~~~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:58:17:{58:23-58:35}: warning: Although the
> value stored to 'tw1' is used in the enclosing expression, the value
> is never actually read from 'tw1'
>     tw3 = tw2 = tw1 = st->twiddles;
>                 ^     ~~~~~~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:55:18:{55:21-55:24}: warning: Value stored
> to 'm3' during its initialization is never read
>     const size_t m3=3*m;
>                  ^  ~~~
> kiss_fft_v1_2_9/kiss_fft.c:54:18:{54:21-54:24}: warning: Value stored
> to 'm2' during its initialization is never read
>     const size_t m2=2*m;
>                  ^  ~~~
> kiss_fft_v1_2_9/kiss_fft.c:117:10:{117:17-117:26}: warning: Value
> stored to 'tw2' is never read
>          tw2 += fstride*2;
>          ^      ~~~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:132:12:{132:10-132:16}: warning: Value
> stored to 'Fout' is never read
>          ++Fout;
>          ~~^~~~
> kiss_fft_v1_2_9/kiss_fft.c:116:10:{116:17-116:24}: warning: Value
> stored to 'tw1' is never read
>          tw1 += fstride;
>          ^      ~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:106:10:{106:14-106:26}: warning: Although
> the value stored to 'tw2' is used in the enclosing expression, the
> value is never actually read from 'tw2'
>      tw1=tw2=st->twiddles;
>          ^   ~~~~~~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:106:6:{106:10-106:26}: warning: Value
> stored to 'tw1' is never read
>      tw1=tw2=st->twiddles;
>      ^   ~~~~~~~~~~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:104:6:{104:13-104:36}: warning: Value
> stored to 'epi3' is never read
>      epi3 = st->twiddles[fstride*m];
>      ^      ~~~~~~~~~~~~~~~~~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:100:19:{100:24-100:27}: warning: Value
> stored to 'm2' during its initialization is never read
>      const size_t m2 = 2*m;
>                   ^    ~~~
> 21 warnings generated.
> _______________________________________________
> cfe-dev mailing list
> [hidden email]
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev


_______________________________________________
cfe-dev mailing list
[hidden email]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev
Reply | Threaded
Open this post in threaded view
|

Re: CSA emits results for kiss_fft

Ted Kremenek
In reply to this post by August Joki
This is now fixed in top of tree LLVM/Clang.

On Aug 10, 2010, at 8:00 PM, August Joki wrote:

> The CSA included with the latest release of Xcode (3.2.3) emits 21
> analyzer results for the latest (v1.2.9) version of kiss_fft. It looks
> like it's having issues with the macros. Is this a known issue?
>
> -August
>
> kiss_fft:
> http://sourceforge.net/projects/kissfft/files/kissfft/v1_2_9/
>
> Analyzer Results:
> kiss_fft_v1_2_9/kiss_fft.c:40:11:{40:9-40:15}: warning: Value stored
> to 'Fout' is never read
>         ++Fout;
>         ~~^~~~
> kiss_fft_v1_2_9/kiss_fft.c:39:11:{39:9-39:16}: warning: Value stored
> to 'Fout2' is never read
>         ++Fout2;
>         ~~^~~~~
> kiss_fft_v1_2_9/kiss_fft.c:36:9:{36:16-36:23}: warning: Value stored
> to 'tw1' is never read
>         tw1 += fstride;
>         ^      ~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:31:5:{31:13-31:21}: warning: Value stored
> to 'Fout2' is never read
>     Fout2 = Fout + m;
>     ^       ~~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:29:20:{29:26-29:38}: warning: Value stored
> to 'tw1' during its initialization is never read
>     kiss_fft_cpx * tw1 = st->twiddles;
>                    ^     ~~~~~~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:88:11:{88:9-88:15}: warning: Value stored
> to 'Fout' is never read
>         ++Fout;
>         ~~^~~~
> kiss_fft_v1_2_9/kiss_fft.c:74:9:{74:16-74:25}: warning: Value stored
> to 'tw3' is never read
>         tw3 += fstride*3;
>         ^      ~~~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:73:9:{73:16-73:25}: warning: Value stored
> to 'tw2' is never read
>         tw2 += fstride*2;
>         ^      ~~~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:72:9:{72:16-72:23}: warning: Value stored
> to 'tw1' is never read
>         tw1 += fstride;
>         ^      ~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:58:11:{58:17-58:35}: warning: Although the
> value stored to 'tw2' is used in the enclosing expression, the value
> is never actually read from 'tw2'
>     tw3 = tw2 = tw1 = st->twiddles;
>           ^     ~~~~~~~~~~~~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:58:5:{58:11-58:35}: warning: Value stored
> to 'tw3' is never read
>     tw3 = tw2 = tw1 = st->twiddles;
>     ^     ~~~~~~~~~~~~~~~~~~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:58:17:{58:23-58:35}: warning: Although the
> value stored to 'tw1' is used in the enclosing expression, the value
> is never actually read from 'tw1'
>     tw3 = tw2 = tw1 = st->twiddles;
>                 ^     ~~~~~~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:55:18:{55:21-55:24}: warning: Value stored
> to 'm3' during its initialization is never read
>     const size_t m3=3*m;
>                  ^  ~~~
> kiss_fft_v1_2_9/kiss_fft.c:54:18:{54:21-54:24}: warning: Value stored
> to 'm2' during its initialization is never read
>     const size_t m2=2*m;
>                  ^  ~~~
> kiss_fft_v1_2_9/kiss_fft.c:117:10:{117:17-117:26}: warning: Value
> stored to 'tw2' is never read
>          tw2 += fstride*2;
>          ^      ~~~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:132:12:{132:10-132:16}: warning: Value
> stored to 'Fout' is never read
>          ++Fout;
>          ~~^~~~
> kiss_fft_v1_2_9/kiss_fft.c:116:10:{116:17-116:24}: warning: Value
> stored to 'tw1' is never read
>          tw1 += fstride;
>          ^      ~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:106:10:{106:14-106:26}: warning: Although
> the value stored to 'tw2' is used in the enclosing expression, the
> value is never actually read from 'tw2'
>      tw1=tw2=st->twiddles;
>          ^   ~~~~~~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:106:6:{106:10-106:26}: warning: Value
> stored to 'tw1' is never read
>      tw1=tw2=st->twiddles;
>      ^   ~~~~~~~~~~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:104:6:{104:13-104:36}: warning: Value
> stored to 'epi3' is never read
>      epi3 = st->twiddles[fstride*m];
>      ^      ~~~~~~~~~~~~~~~~~~~~~~~
> kiss_fft_v1_2_9/kiss_fft.c:100:19:{100:24-100:27}: warning: Value
> stored to 'm2' during its initialization is never read
>      const size_t m2 = 2*m;
>                   ^    ~~~
> 21 warnings generated.
> _______________________________________________
> cfe-dev mailing list
> [hidden email]
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev

_______________________________________________
cfe-dev mailing list
[hidden email]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev