r210333 - Quieting a false-positive which was causing the sanitizer bots to go red.

[Kostya Serebryany]

Just to make sure, did you mean this:
http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/10289/steps/bootstrap%20clang/logs/stdio
?
This is clang build using -DLLVM_ENABLE_WERROR=ON and report comes from
clang, sanitizers are not involved, right?

If you think this is a false positive, shouldn't we fix it in clang?


On Fri, Jun 6, 2014 at 5:39 PM, Aaron Ballman <aaron at aaronballman.com>
wrote:

> Author: aaronballman
> Date: Fri Jun  6 08:39:32 2014
> New Revision: 210333
>
> URL: http://llvm.org/viewvc/llvm-project?rev=210333&view=rev
> Log:
> Quieting a false-positive which was causing the sanitizer bots to go red.
>
> Modified:
>     cfe/trunk/lib/Sema/SemaStmtAttr.cpp
>
> Modified: cfe/trunk/lib/Sema/SemaStmtAttr.cpp
> URL:
> http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaStmtAttr.cpp?rev=210333&r1=210332&r2=210333&view=diff
>
> ==============================================================================
> --- cfe/trunk/lib/Sema/SemaStmtAttr.cpp (original)
> +++ cfe/trunk/lib/Sema/SemaStmtAttr.cpp Fri Jun  6 08:39:32 2014
> @@ -69,7 +69,7 @@ static Attr *handleLoopHintAttr(Sema &S,
>            .Case("interleave_count", LoopHintAttr::InterleaveCount)
>            .Default(LoopHintAttr::Vectorize);
>
> -  int ValueInt;
> +  int ValueInt = 0;
>    if (Option == LoopHintAttr::Vectorize || Option ==
> LoopHintAttr::Interleave) {
>      if (!ValueInfo) {
>        S.Diag(ValueLoc->Loc, diag::err_pragma_loop_invalid_keyword)
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140606/c63c6a24/attachment.html>