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

Reply via email to