On Fri, Apr 01, 2022 at 07:38:59PM +0800, Xi Ruoyao wrote: > v1 -> v2: > > * "int has_zero_width_bf" -> "bool has_cxx_zero_width_bf". "int" to > "bool" because the value is 0/1 only. Add "cxx" because it only > indicates C++ zero-width bit-fields (not those bit-fields from C). > > * Coding style fix. > > * Rewrite mips_return_in_msb so mips_fpr_return_fields is not called > unnecessarily. > > * "retcon" -> "change". > > gcc/ > PR target/102024 > * mips.cc (mips_fpr_return_fields): Detect C++ zero-width > bit-fields and set up an indicator. > (mips_return_in_msb): Adapt for mips_fpr_return_fields change. > (mips_function_value_1): Diagnose when the presense of a C++ > zero-width bit-field changes function returning in GCC 12. > > gcc/testsuite/ > PR target/102024 > * g++.target/mips/mips.exp: New test supporting file. > * g++.target/mips/pr102024-1.C: New test.
LGTM, thanks. Jakub