On 05/19/2016 01:16 AM, Jakub Jelinek wrote:
On Wed, May 18, 2016 at 05:13:25PM -0500, Segher Boessenkool wrote:
On Wed, May 18, 2016 at 01:35:16PM -0500, Segher Boessenkool wrote:
On Wed, May 18, 2016 at 11:20:29AM -0700, H.J. Lu wrote:
* function.c (make_split_prologue_seq, make_prologue_seq,
make_epilogue_seq): New functions, factored out from...
(thread_prologue_and_epilogue_insns): Here.
It breaks x86:
Are you sure it is this patch causing it? As noted, it was tested on x86.
I am pretty sure. How did you test it on x86?
"make -k check". I'll test 32-bit now.
Actually, it also fails on 64 bit. It passed my testing because it does
not fail together with patch 3/3, and does not fail on powerpc at all.
If 3/3 isn't approved soon, can you please revert the problematic commit
until it is if that patch can't work right on its own and needs the other
patch too)? The trunk is in terrible state right now at least on
x86_64/i686-linux, various tests hang forever (e.g. some cleanup-* tests)
and there are hundreds of failures, making it impossible to do proper
regression testing.
FWIW, I'm still looking at 3/3.
jeff
