Re: [COMMITED][AArch64] Fix PR79041

2017-07-26 Thread Wilco Dijkstra
Andreas Schwab wrote: > That fails in ILP32 mode. Well -mabi-ilp32 and -mcmodel=large make no sense at all, that should really give an error... I've committed a patch to trunk and GCC7 to disable this test with ILP32. Wilco

Re: [COMMITED][AArch64] Fix PR79041

2017-07-25 Thread Andreas Schwab
On Jul 25 2017, Wilco Dijkstra wrote: > diff --git a/gcc/testsuite/gcc.target/aarch64/pr79041-2.c > b/gcc/testsuite/gcc.target/aarch64/pr79041-2.c > new file mode 100644 > index > ..cd34fbab85a92d00cba7091d4146deaaf3a862a9 > --- /dev/null > +++ b/gcc/test

Re: [COMMITED][AArch64] Fix PR79041

2017-07-25 Thread Wilco Dijkstra
Ramana Radhakrishnan > > BZ suggests that this affects GCC 6 but GCC 7 is fixed ? Should there > be a backport to GCC 6 as well ? > > Can you please keep BZ up to date please ? GCC7 was affected as well, the example in PR79041 didn't trigger in GCC7, my patch has a better example that does show t

Re: [COMMITED][AArch64] Fix PR79041

2017-07-25 Thread Ramana Radhakrishnan
On Tue, Jul 25, 2017 at 1:13 PM, Wilco Dijkstra wrote: > James Greenhalgh wrote: >> >> OK. Either like this, or with the conditions swapped around as Yvan >> suggested to make backporting easier. > > I swapped the conditions around, not sure whether it helps... > Also I needed an additional scan-a

Re: [COMMITED][AArch64] Fix PR79041

2017-07-25 Thread Wilco Dijkstra
James Greenhalgh wrote: > > OK. Either like this, or with the conditions swapped around as Yvan > suggested to make backporting easier. I swapped the conditions around, not sure whether it helps... Also I needed an additional scan-assembler, this was committed to trunk and GCC7: diff --git a/gcc/