From 2bbc19c1c6cd60b9c56750d7b03311c47a88b748 Mon Sep 17 00:00:00 2001 From: David Blaikie Date: Thu, 21 Feb 2013 06:05:57 +0000 Subject: [PATCH] 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 --- include/llvm/ADT/Optional.h | 2 +- include/llvm/Support/Compiler.h | 10 ++++++++++ 2 files changed, 11 insertions(+), 1 deletion(-) 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 -- 2.34.1