From: David Blaikie Date: Thu, 21 Feb 2013 06:05:57 +0000 (+0000) Subject: Make Optional's operator bool 'explicit' in C++11 X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=2bbc19c1c6cd60b9c56750d7b03311c47a88b748;p=oota-llvm.git Make Optional's operator bool 'explicit' in C++11 Provides a general way to add 'explicit' for conversion operators (a no-op when compiling as C++98). git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@175723 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/include/llvm/ADT/Optional.h b/include/llvm/ADT/Optional.h index 844e3090500..c5dc29946cb 100644 --- a/include/llvm/ADT/Optional.h +++ b/include/llvm/ADT/Optional.h @@ -86,7 +86,7 @@ public: const T& getValue() const LLVM_LVALUE_FUNCTION { assert(hasVal); return *getPointer(); } T& getValue() LLVM_LVALUE_FUNCTION { assert(hasVal); return *getPointer(); } - operator bool() const { return hasVal; } + LLVM_EXPLICIT operator bool() const { return hasVal; } bool hasValue() const { return hasVal; } const T* operator->() const { return getPointer(); } T* operator->() { return getPointer(); } diff --git a/include/llvm/Support/Compiler.h b/include/llvm/Support/Compiler.h index adc0ce2ab14..25f42a98e7d 100644 --- a/include/llvm/Support/Compiler.h +++ b/include/llvm/Support/Compiler.h @@ -341,4 +341,14 @@ # define LLVM_IS_UNALIGNED_ACCESS_FAST 0 #endif +/// \macro LLVM_EXPLICIT +/// \brief Expands to explicit on compilers which support explicit conversion +/// operators. Otherwise expands to nothing. +#if (__has_feature(cxx_explicit_conversions) \ + || defined(__GXX_EXPERIMENTAL_CXX0X__)) +#define LLVM_EXPLICIT explicit +#else +#define LLVM_EXPLICIT +#endif + #endif