diff --git a/gcc/cp/constexpr.c b/gcc/cp/constexpr.c index 7d96d577d84d..c94674471b7b 100644 --- a/gcc/cp/constexpr.c +++ b/gcc/cp/constexpr.c @@ -5582,20 +5582,17 @@ cxx_eval_increment_expression (const constexpr_ctx *ctx, tree t, /* Storing the modified value. */ tree store = build2_loc (cp_expr_loc_or_loc (t, input_location), MODIFY_EXPR, type, op, mod); - cxx_eval_constant_expression (ctx, store, - true, non_constant_p, overflow_p); + mod = cxx_eval_constant_expression (ctx, store, lval, + non_constant_p, overflow_p); ggc_free (store); + if (*non_constant_p) + return t; /* And the value of the expression. */ if (code == PREINCREMENT_EXPR || code == PREDECREMENT_EXPR) - { - /* Prefix ops are lvalues. */ - if (lval) - return op; - else - /* But we optimize when the caller wants an rvalue. */ - return mod; - } + /* Prefix ops are lvalues, but the caller might want an rvalue; + lval has already been taken into account in the store above. */ + return mod; else /* Postfix ops are rvalues. */ return val; diff --git a/gcc/testsuite/g++.dg/cpp2a/constexpr-99287.C b/gcc/testsuite/g++.dg/cpp2a/constexpr-99287.C new file mode 100644 index 000000000000..c9c2ac2f7c25 --- /dev/null +++ b/gcc/testsuite/g++.dg/cpp2a/constexpr-99287.C @@ -0,0 +1,61 @@ +// PR c++/99287 +// { dg-do compile { target c++20 } } + +namespace std { +struct source_location { + static consteval source_location + current(const void *__p = __builtin_source_location()) { + source_location __ret; + __ret._M_impl = static_cast(__p); + return __ret; + } + constexpr const char *function_name() { + return _M_impl ? _M_impl->_M_function_name : ""; + } + struct __impl { + const char *_M_file_name; + const char *_M_function_name; + unsigned _M_line; + unsigned _M_column; + } const *_M_impl; +}; +struct char_traits { + static constexpr long length(const char *__s) { + return __builtin_strlen(__s); + } +}; +template +class basic_string_view { +public: + using traits_type = _Traits; + using size_type = unsigned long; + constexpr basic_string_view(const _CharT *__str) + : _M_len{traits_type::length(__str)}, _M_str{__str} {} + constexpr size_type find(const _CharT *, size_type, size_type) const noexcept; + constexpr size_type find(_CharT *__str) { + long __trans_tmp_1 = traits_type::length(__str); + return find(__str, 0, __trans_tmp_1); + } + long _M_len; + const _CharT *_M_str; +}; +using string_view = basic_string_view; +template +constexpr unsigned long +basic_string_view<_CharT, _Traits>::find(const _CharT *__str, size_type, + size_type __n) const noexcept { + int __trans_tmp_2; + const _CharT *__first = _M_str; + size_type __len = _M_len; + while (__len >= __n) { + __trans_tmp_2 = __builtin_memcmp(__first, __str, __n); + if (__trans_tmp_2 == 0) + return __first - _M_str; + __len = _M_str - ++__first; + } +} +} // namespace std +template consteval auto f() { + return std::string_view{std::source_location::current().function_name()}; +} +int main() { constexpr auto s = f().find("int"); }