"Richard Earnshaw (lists)" <richard.earns...@arm.com> writes: > On 09/01/2025 14:50, Christophe Lyon wrote: >> The previous fix only worked for C, for C++ we need to add more >> information to the underlying type so that >> finish_class_member_access_expr accepts it. >> >> We use the same logic as in aarch64's register_tuple_type for AdvSIMD >> tuples. >> >> This patch makes gcc.target/arm/mve/intrinsics/pr118332.c pass in C++ >> mode. >> >> gcc/ChangeLog: >> >> PR target/118332 >> * config/arm/arm-mve-builtins.cc (wrap_type_in_struct): Delete. >> (register_type_decl): Delete. >> (register_builtin_tuple_types): Use >> lang_hooks.types.simulate_record_decl. > > Much nicer. > > OK, but please give Richard S 24 hours to comment.
Yeah, LGTM too. Richard