Am 27.07.2014 13:59, schrieb pins...@gmail.com:
> 
> 
>> On Jul 27, 2014, at 4:53 AM, Alan Modra <amo...@gmail.com> wrote:
>>
>>> On Sun, Jul 27, 2014 at 07:16:07PM +0930, Alan Modra wrote:
>>>> On Sat, Jul 26, 2014 at 01:45:12PM +0200, Matthias Klose wrote:
>>>> Am 17.07.2014 02:41, schrieb Ulrich Weigand:
>>>>> Hello,
>>>>>
>>>>> this is the variant intended for the 4.8/4.9 branches of the patch:
>>>>> https://gcc.gnu.org/ml/gcc-patches/2014-07/msg01072.html
>>>>>
>>>>> As discussed, it does *not* actually change ABI, but only warn when
>>>>> encountering a situation where the ABI will change in a future GCC.
>>>>> (Avoiding the specific term "GCC 4.10" here since I'm not certain
>>>>> whether the next GCC release will in fact be called that ...)
>>>>>
>>>>> Tested on powerpc64-linux and powerpc64le-linux; also verified using
>>>>> the ABI compat suite (against an unpatched GCC) that this patch does
>>>>> not change the ABI.
>>>>
>>>> this causes PR libobjc/61920, link failures with -lobjc.
>>>
>>> Try this.
>>>
>>> Index: libobjc/encoding.c
>>> ===================================================================
>>> --- libobjc/encoding.c    (revision 213090)
>>> +++ libobjc/encoding.c    (working copy)
>>> @@ -192,6 +192,7 @@
>>>    ? MAX (MAX (COMPUTED, SPECIFIED), 64)                \
>>>    : MAX (COMPUTED, SPECIFIED));})
>>>
>>> +#define rs6000_special_adjust_field_align_p false
>>>
>>> /* Skip a variable name, enclosed in quotes (").  */
>>> static inline
>>
>> Blah, that won't work of course.  The macro needs to take two
>> parameters.
>>
>> #define rs6000_special_adjust_field_align_p(FIELD, COMPUTED) false
> 
> This is pre-approved if it works.  I really should finish off the branch I 
> started years ago :). 

#define rs6000_special_adjust_field_align_p(FIELD, COMPUTED) 0

is what succeeds for me. false is not defined for ObjC. Checked in on the trunk
and the branches.

these are still regressions, because the new warnings trigger on these test 
cases:

Running
/home/doko/gcc/gcc-4.9-4.9.1/src/gcc/testsuite/objc.dg/gnu-encoding/gnu-encoding.exp
...
FAIL: objc.dg-struct-layout-encoding-1/t025_main.m execution test
FAIL: objc.dg-struct-layout-encoding-1/t027_main.m execution test
FAIL: objc.dg-struct-layout-encoding-1/t028_main.m execution test
FAIL: objc.dg-struct-layout-encoding-1/t029_main.m execution test
FAIL: objc.dg-struct-layout-encoding-1/t030_main.m execution test
FAIL: objc.dg-struct-layout-encoding-1/t031_main.m execution test

No regressions in the obj-c++ tests.

  Matthias

Reply via email to