"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

Reply via email to