[commit, 4.8/4.9, libobjc] Match branch ABI (Re: [PATCH, rs6000, 4.8/4.9] Fix alignment of non-Altivec vector struct fields)

2014-07-28 Thread Ulrich Weigand
> Matthias Klose wrote: > > > #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. > > Since we didn't backport the actual ABI change to the branches, only the > warning,

Re: [PATCH, rs6000, 4.8/4.9] Fix alignment of non-Altivec vector struct fields

2014-07-28 Thread Ulrich Weigand
Matthias Klose wrote: > #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. Since we didn't backport the actual ABI change to the branches, only the warning, I think it would be

Re: [PATCH, rs6000, 4.8/4.9] Fix alignment of non-Altivec vector struct fields

2014-07-27 Thread Alan Modra
On Sun, Jul 27, 2014 at 03:16:03PM +0200, Matthias Klose wrote: > >> 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

Re: [PATCH, rs6000, 4.8/4.9] Fix alignment of non-Altivec vector struct fields

2014-07-27 Thread Matthias Klose
Am 27.07.2014 13:59, schrieb pins...@gmail.com: > > >> On Jul 27, 2014, at 4:53 AM, Alan Modra 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: > He

Re: [PATCH, rs6000, 4.8/4.9] Fix alignment of non-Altivec vector struct fields

2014-07-27 Thread pinskia
> On Jul 27, 2014, at 4:53 AM, Alan Modra 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/

Re: [PATCH, rs6000, 4.8/4.9] Fix alignment of non-Altivec vector struct fields

2014-07-27 Thread Alan Modra
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/g

Re: [PATCH, rs6000, 4.8/4.9] Fix alignment of non-Altivec vector struct fields

2014-07-27 Thread Alan Modra
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* ac

Re: [PATCH, rs6000, 4.8/4.9] Fix alignment of non-Altivec vector struct fields

2014-07-26 Thread Matthias Klose
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

Re: [PATCH, rs6000, 4.8/4.9] Fix alignment of non-Altivec vector struct fields

2014-07-18 Thread Jakub Jelinek
On Fri, Jul 18, 2014 at 12:01:21PM -0400, David Edelsohn wrote: > This patch is okay with me if okay with the Release Managers. Ok. Jakub

Re: [PATCH, rs6000, 4.8/4.9] Fix alignment of non-Altivec vector struct fields

2014-07-18 Thread David Edelsohn
This patch is okay with me if okay with the Release Managers. Thanks, David On Wed, Jul 16, 2014 at 8:41 PM, Ulrich Weigand wrote: > 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

[PATCH, rs6000, 4.8/4.9] Fix alignment of non-Altivec vector struct fields

2014-07-16 Thread 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 "