Re: [PATCH, rs6000] Fix PR target/80210: ICE in extract_insn

2017-09-12 Thread Peter Bergner
On 9/9/17 3:44 AM, Andreas Schwab wrote: > On Sep 08 2017, Peter Bergner wrote: > >> The following patch fixes the problem I saw on Linux and bootstraps and >> regtests >> with no regressions on LE and BE (running testsuite in both 32-bit and 64-bit >> modes). I was waiting to submit this until

Re: [PATCH, rs6000] Fix PR target/80210: ICE in extract_insn

2017-09-09 Thread Andreas Schwab
On Sep 08 2017, Peter Bergner wrote: > The following patch fixes the problem I saw on Linux and bootstraps and > regtests > with no regressions on LE and BE (running testsuite in both 32-bit and 64-bit > modes). I was waiting to submit this until David had a chance to verify this > fixes the pr

Re: [PATCH, rs6000] Fix PR target/80210: ICE in extract_insn

2017-09-08 Thread Peter Bergner
On 9/8/17 2:11 PM, Andreas Schwab wrote: > Executing on host: /daten/gcc/gcc-20170907/Build/gcc/xgcc > -B/daten/gcc/gcc-20170907/Build/gcc/ > /daten/gcc/gcc-20170907/gcc/testsuite/gcc.target/powerpc/pr80125.c -m32 > -fno-diagnostics-show-caret -fdiagnostics-color=never -O2 -maltivec -S -o

Re: [PATCH, rs6000] Fix PR target/80210: ICE in extract_insn

2017-09-08 Thread Andreas Schwab
Executing on host: /daten/gcc/gcc-20170907/Build/gcc/xgcc -B/daten/gcc/gcc-20170907/Build/gcc/ /daten/gcc/gcc-20170907/gcc/testsuite/gcc.target/powerpc/pr80125.c -m32 -fno-diagnostics-show-caret -fdiagnostics-color=never -O2 -maltivec -S -o pr80125.s(timeout = 300) FAIL: gcc.target/p

Re: [PATCH, rs6000] Fix PR target/80210: ICE in extract_insn

2017-08-23 Thread Peter Bergner
On 8/18/17 9:19 PM, Peter Bergner wrote: > On 8/18/17 6:27 PM, Segher Boessenkool wrote: >> On Thu, Aug 17, 2017 at 07:02:14PM -0500, Peter Bergner wrote: >>> This is also broken in GCC 7, GCC 6 and GCC 5. Ok for those after this >>> has been on trunk for a little while and assuming testing passes

Re: [PATCH, rs6000] Fix PR target/80210: ICE in extract_insn

2017-08-18 Thread Peter Bergner
On 8/18/17 6:27 PM, Segher Boessenkool wrote: > On Thu, Aug 17, 2017 at 07:02:14PM -0500, Peter Bergner wrote: >> This is also broken in GCC 7, GCC 6 and GCC 5. Ok for those after this >> has been on trunk for a little while and assuming testing passes? > > Okay for trunk and all branches. Thank

Re: [PATCH, rs6000] Fix PR target/80210: ICE in extract_insn

2017-08-18 Thread Segher Boessenkool
On Thu, Aug 17, 2017 at 07:02:14PM -0500, Peter Bergner wrote: > PR target/80210 exposes a problem in rs6000_set_current_function() where > is fails to correctly clear the rs6000_previous_fndecl cache correctly. > With the test case, we notice that rs6000_previous_fndecl is set (when it > shouldn't

[PATCH, rs6000] Fix PR target/80210: ICE in extract_insn

2017-08-17 Thread Peter Bergner
PR target/80210 exposes a problem in rs6000_set_current_function() where is fails to correctly clear the rs6000_previous_fndecl cache correctly. With the test case, we notice that rs6000_previous_fndecl is set (when it shouldn't be) and we end up restoring options from it. In this case, we end up