diff --git a/clang/lib/AST/ByteCode/Compiler.cpp b/clang/lib/AST/ByteCode/Compiler.cpp index 9a52dd4105437..13b8a3b47add6 100644 --- a/clang/lib/AST/ByteCode/Compiler.cpp +++ b/clang/lib/AST/ByteCode/Compiler.cpp @@ -2309,29 +2309,36 @@ bool Compiler::VisitAbstractConditionalOperator( return visitChildExpr(FalseExpr); } + bool IsBcpCall = false; + if (const auto *CE = dyn_cast(Condition->IgnoreParenCasts()); + CE && CE->getBuiltinCallee() == Builtin::BI__builtin_constant_p) { + IsBcpCall = true; + } + LabelTy LabelEnd = this->getLabel(); // Label after the operator. LabelTy LabelFalse = this->getLabel(); // Label for the false expr. + if (IsBcpCall) { + if (!this->emitStartSpeculation(E)) + return false; + } + if (!this->visitBool(Condition)) return false; - if (!this->jumpFalse(LabelFalse)) return false; - if (!visitChildExpr(TrueExpr)) return false; - if (!this->jump(LabelEnd)) return false; - this->emitLabel(LabelFalse); - if (!visitChildExpr(FalseExpr)) return false; - this->fallthrough(LabelEnd); this->emitLabel(LabelEnd); + if (IsBcpCall) + return this->emitEndSpeculation(E); return true; } diff --git a/clang/test/AST/ByteCode/builtin-constant-p.cpp b/clang/test/AST/ByteCode/builtin-constant-p.cpp index b309fa8296889..ed9e606ed16aa 100644 --- a/clang/test/AST/ByteCode/builtin-constant-p.cpp +++ b/clang/test/AST/ByteCode/builtin-constant-p.cpp @@ -59,13 +59,10 @@ template constexpr bool bcp(T t) { } constexpr intptr_t ptr_to_int(const void *p) { - return __builtin_constant_p(1) ? (intptr_t)p : (intptr_t)p; // expected-note {{cast that performs the conversions of a reinterpret_cast}} + return __builtin_constant_p(1) ? (intptr_t)p : (intptr_t)p; } -/// This is from test/SemaCXX/builtin-constant-p.cpp, but it makes no sense. -/// ptr_to_int is called before bcp(), so it fails. GCC does not accept this either. -static_assert(bcp(ptr_to_int("foo"))); // expected-error {{not an integral constant expression}} \ - // expected-note {{in call to}} +static_assert(bcp(ptr_to_int("foo"))); constexpr bool AndFold(const int &a, const int &b) { return __builtin_constant_p(a && b);