On Tue, Jun 07, 2016 at 12:07:03PM +0100, James Greenhalgh wrote: > On Fri, Jan 22, 2016 at 05:16:00PM +0000, Alan Lawrence wrote: > > > > On 21/01/16 17:23, Alan Lawrence wrote: > > > On 18/01/16 17:10, Eric Botcazou wrote: > > >> > > >> Could you post the list of files that differ? How do they differ > > >> exactly? > > > > > > Hmmm. Well, I definitely had this failing to bootstrap once. I repeated > > > that, to > > > try to identify exactly what the differences were....and it succeeded > > > even with > > > my pre-AAPCS64-update host compiler. So, this is probably a false alarm; > > > I'm > > > bootstrapping again, after a rebase, to make sure... > > > > > > --Alan > > > > Ok, rebased onto a more recent build, and bootstrapping with Ada posed no > > problems. Sorry for the noise. > > > > However, I had to drop the assert that TYPE_FIELDS was non-null because of > > some > > C++ testcases. > > > > Is this version OK for trunk? > > Now that we're in GCC7, this version of the patch is OK for trunk. > > From my reading of Richard's AAPCS update, this patch implements the > rules as required. > > I'll give this a day for any last minute comments from Richard/Marcus, > then commit this on your behalf tomorrow.
I've now committed this on Alan's behalf as revisions r237224 (this patch) and r237225 (the tests) respectively. Thanks, James