On 2/25/26 5:13 AM, Marek Polacek wrote:
Bootstrapped/regtested on x86_64-pc-linux-gnu, ok for trunk?
-- >8 --
Here when asking for the members of "^^Test const" we found nothing.
The reason is that members_of_representable_p checks whether
CP_DECL_CONTEXT != c, but if c is a type, it doesn't look at its
TYPE_MAIN_VARIANT. Fixed as per Jakub's suggestion in the PR.
PR c++/124215
gcc/cp/ChangeLog:
* reflect.cc (class_members_of): Pass TYPE_MAIN_VARIANT of R to
members_of_representable_p.
Would it make sense to r = TYPE_MAIN_VARIANT at the top of the function
instead of doing it in the loop?
gcc/testsuite/ChangeLog:
* g++.dg/reflect/members_of9.C: New test.
Co-authored-by: Jakub Jelinek <[email protected]>
---
gcc/cp/reflect.cc | 2 +-
gcc/testsuite/g++.dg/reflect/members_of9.C | 17 +++++++++++++++++
2 files changed, 18 insertions(+), 1 deletion(-)
create mode 100644 gcc/testsuite/g++.dg/reflect/members_of9.C
diff --git a/gcc/cp/reflect.cc b/gcc/cp/reflect.cc
index 522b7c06a29..906dc86af4f 100644
--- a/gcc/cp/reflect.cc
+++ b/gcc/cp/reflect.cc
@@ -6600,7 +6600,7 @@ class_members_of (location_t loc, const constexpr_ctx
*ctx, tree r,
--function_depth;
}
}
- if (members_of_representable_p (r, m))
+ if (members_of_representable_p (TYPE_MAIN_VARIANT (r), m))
{
if (kind == METAFN_STATIC_DATA_MEMBERS_OF
&& eval_is_variable (m, REFLECT_UNDEF) != boolean_true_node)
diff --git a/gcc/testsuite/g++.dg/reflect/members_of9.C
b/gcc/testsuite/g++.dg/reflect/members_of9.C
new file mode 100644
index 00000000000..9d07182953b
--- /dev/null
+++ b/gcc/testsuite/g++.dg/reflect/members_of9.C
@@ -0,0 +1,17 @@
+// PR c++/124215
+// { dg-do compile { target c++26 } }
+// { dg-additional-options "-freflection" }
+
+#include <meta>
+
+struct Test {
+ int a;
+ char b;
+};
+
+consteval auto number_of_members(std::meta::info ty) -> size_t {
+ return nonstatic_data_members_of(ty,
std::meta::access_context::current()).size();
+}
+
+static_assert(number_of_members(^^Test) == 2);
+static_assert(number_of_members(^^Test const) == 2);
base-commit: b67918fddab42c434c10bedff6c210c55ed907a0