Re: r306809 - Ambiguity might be also uninitialized. Use llvm::Optional.

2017-07-10 Thread David Blaikie via cfe-commits
Which compiler/what warning was flagging this? It doesn't look like Clang builds by default with -Wconditional-uninitialized on, so I'm surprised if Clang was diagnosing anything here. In any case - it's probably better to omit the "WasAmbiguous" flag. Optional already has a flag that says whethe

r306809 - Ambiguity might be also uninitialized. Use llvm::Optional.

2017-06-30 Thread Vassil Vassilev via cfe-commits
Author: vvassilev Date: Fri Jun 30 02:25:43 2017 New Revision: 306809 URL: http://llvm.org/viewvc/llvm-project?rev=306809&view=rev Log: Ambiguity might be also uninitialized. Use llvm::Optional. Modified: cfe/trunk/include/clang/Sema/Lookup.h Modified: cfe/trunk/include/clang/Sema/Lookup.h