On Tue, Sep 12, 2017 at 9:49 AM, Steve Ellcey <sell...@cavium.com> wrote: > On Tue, 2017-09-12 at 09:39 -0700, Ian Lance Taylor wrote: >> On Tue, Sep 12, 2017 at 3:59 AM, Wilco Dijkstra <Wilco.Dijkstra@arm.c >> om> wrote: >> > >> > Steve Ellcey wrote: >> > > >> > > This patch fixes the ttest failures on aarch64 by adding >> > > AM_CFLAGS to >> > > the test options, like btest already does and as Wilco says works >> > > for >> > > him in Comment #4 of the bug report. >> > Thanks for picking this up, this looks OK. >> > >> > > >> > > Tested by me on aarch64. Ok to checkin? >> > This counts as an obvious fix, so you can commit it. >> Wait, what? This patch is at best incomplete. Makefile.in is a >> generated file. You need to change Makefile.am and re-run automake. >> >> Ian > > OK, here is the new patch that I will checkin. I verified that after > running automake on Makefile.am, the Makefile.in I got was identical > to what I checked in earlier.
Thanks for taking care of that. Ian