Re: [PATCH, rs6000] Follow-on fix for PR target/80210: ICE in extract_insn

2017-12-14 Thread Peter Bergner
On 10/2/17 12:10 PM, Segher Boessenkool wrote: > On Mon, Oct 02, 2017 at 12:00:55PM -0500, Peter Bergner wrote: >> On 9/29/17 5:31 PM, Segher Boessenkool wrote: >>> Looks great to me, please commit. But hold off until Monday please, it >>> will interfere with testing otherwise. >> >> Ok,

Re: [PATCH, rs6000] Follow-on fix for PR target/80210: ICE in extract_insn

2017-10-02 Thread Segher Boessenkool
On Mon, Oct 02, 2017 at 12:00:55PM -0500, Peter Bergner wrote: > On 9/29/17 5:31 PM, Segher Boessenkool wrote: > >> +/* PowerPC 64-bit LE requires at least ISA 2.07. */ > >> +const char *default_cpu = ((!TARGET_POWERPC64) > >> + ? "powerpc" > >> +

Re: [PATCH, rs6000] Follow-on fix for PR target/80210: ICE in extract_insn

2017-10-02 Thread Peter Bergner
On 9/29/17 5:31 PM, Segher Boessenkool wrote: >> + /* PowerPC 64-bit LE requires at least ISA 2.07. */ >> + const char *default_cpu = ((!TARGET_POWERPC64) >> + ? "powerpc" >> + : ((BYTES_BIG_ENDIAN) >> +

Re: [PATCH, rs6000] Follow-on fix for PR target/80210: ICE in extract_insn

2017-09-29 Thread Segher Boessenkool
Hi Peter, On Thu, Sep 28, 2017 at 09:26:00PM -0500, Peter Bergner wrote: > This patch fixes two new issues exposed (but not caused) by the original > test case added for PR80210 as well as a modified version of that test case. [ .. snip ... ] This all seems correct (but it is so complex that

[PATCH, rs6000] Follow-on fix for PR target/80210: ICE in extract_insn

2017-09-28 Thread Peter Bergner
This patch fixes two new issues exposed (but not caused) by the original test case added for PR80210 as well as a modified version of that test case. The first problem is that the test case pr80210.c ICEs on 32-bit compiles that do not pass either an explicit or implicit -mcpu=... option. I did