On Tue, Jul 15, 2025 at 6:06 AM Jakub Jelinek <ja...@redhat.com> wrote: > > On Tue, Jul 15, 2025 at 08:21:50AM -0400, Jason Merrill wrote: > > Given the above that seems rather unlikely, but I suppose it's fine if you > > want to do it that way. The patch is OK either way. > > Committed just the v2 patch. I can test your patch next with other patches, > or do you want to test/commit it yourself?
Just an FYI, this broke aarch64 build since libgcc has -Werror on there and we get warnings now in libbid code: ``` ../../../gcc/libgcc/config/libbid/bid_binarydecimal.c: In function ‘__binary32_to_bid128’: ../../../gcc/libgcc/config/libbid/bid_binarydecimal.c:130:31: error: variable ‘c3’ set but not used [-Werror=unused-but-set-variable=] 130 | { unsigned long long c0,c1,c2,c3; \ | ^~ ../../../gcc/libgcc/config/libbid/bid_binarydecimal.c:146842:5: note: in expansion of macro ‘__mul_10x256_to_256’ 146842 | __mul_10x256_to_256 (z.w[5], z.w[4], z.w[3], z.w[2], z.w[5], z.w[4], | ^~~~~~~~~~~~~~~~~~~ ``` https://builder.sourceware.org/buildbot/#/builders/311/builds/1483 I will see if there is anything simple to be done in the next few hours. Thanks, Andrew > > Jakub >