This change broke clang/ubsan bot.
http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-fast/builds/6047/steps/check-clang%20ubsan/logs/stdio

It looks like the value you are initializing SavedAK with may itself be
uninitialized? I see a few constructors that do not mention it.


On Thu, Jun 29, 2017 at 9:08 AM, Vassil Vassilev via cfe-commits <
cfe-commits@lists.llvm.org> wrote:

> Author: vvassilev
> Date: Thu Jun 29 09:08:10 2017
> New Revision: 306692
>
> URL: http://llvm.org/viewvc/llvm-project?rev=306692&view=rev
> Log:
> Initialize variable and silence potentially uninitialized warning.
>
> Patch by Liza Sakellari!
>
> Modified:
>     cfe/trunk/include/clang/Sema/Lookup.h
>
> Modified: cfe/trunk/include/clang/Sema/Lookup.h
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/
> clang/Sema/Lookup.h?rev=306692&r1=306691&r2=306692&view=diff
> ============================================================
> ==================
> --- cfe/trunk/include/clang/Sema/Lookup.h (original)
> +++ cfe/trunk/include/clang/Sema/Lookup.h Thu Jun 29 09:08:10 2017
> @@ -465,10 +465,9 @@ public:
>          Paths = nullptr;
>        }
>      } else {
> -      AmbiguityKind SavedAK;
> +      AmbiguityKind SavedAK = Ambiguity;
>        bool WasAmbiguous = false;
>        if (ResultKind == Ambiguous) {
> -        SavedAK = Ambiguity;
>          WasAmbiguous = true;
>        }
>        ResultKind = Found;
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits@lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to