Whoops, sorry for that mail - I typoed gdb-patches to gcc-patches.
On 20/02/16 14:56, Marcin Kościelnicki wrote:
While groveling through the old PPC64 tracepoint support patch, I've
noticed a few target dependencies in the testsuite that both me and
Antoine missed for s390 and ARM tracepoints
@@ -1,3 +1,14 @@
+2016-02-19 Marcin Kościelnicki <koria...@0x04.net>
+
+ * gdb.trace/entry-values.exp: Move call_insn setting to
+ trace-support.exp.
+ * gdb.trace/ftrace.exp: Move arg0exp setting to trace-support.exp.
+ * gdb.trace/mi-trace-unavailab
/testsuite/ChangeLog
@@ -1,5 +1,9 @@
2016-02-19 Marcin Kościelnicki <koria...@0x04.net>
+ * gdb.trace/ftrace.exp: Remove unnecessary target check.
+
+2016-02-19 Marcin Kościelnicki <koria...@0x04.net>
+
* gdb.trace/entry-values.exp: Surround $call_insn with '\y'.
og
@@ -1,5 +1,10 @@
2016-02-19 Marcin Kościelnicki <koria...@0x04.net>
+ * gdb.trace/entry-values.exp: Surround $call_insn with '\y'.
+ * lib/trace-support.exp: Change x86_64 call_insn to 'callq'.
+
+2016-02-19 Marcin Kościelnicki <koria...@0x04.net>
+
* gdb.
On 21/01/16 14:03, Marcin Kościelnicki wrote:
On TARGET_CPU_ZARCH && !TARGET_64BIT, we can use a similiar lean mcount
call sequence to TARGET_64BIT. The longer sequences are now used only
on deprecated g5/g6 CPUs.
Tested on s390-ibm-linux-gnu on RHEL 7.2.
gcc/ChangeLog:
* con
On 15/02/16 11:21, Andreas Krebbel wrote:
On 02/14/2016 05:01 PM, Marcin Kościelnicki wrote:
libgcc/ChangeLog:
* config.host: Use t-stack and t-stack-s390 for s390*-*-linux.
* config/s390/morestack.S: New file.
* config/s390/t-stack-s390: New file.
* generic
+++ b/gcc/ChangeLog
@@ -1,3 +1,33 @@
+2016-02-14 Marcin Kościelnicki <koria...@0x04.net>
+
+ * common/config/s390/s390-common.c (s390_supports_split_stack):
+ New function.
+ (TARGET_SUPPORTS_SPLIT_STACK): New macro.
+ * config/s390/s390-protos.
On 04/02/16 13:44, Marcin Kościelnicki wrote:
On 03/02/16 18:27, Ulrich Weigand wrote:
Marcin Kościelnicki wrote:
libgcc/ChangeLog:
* config.host: Use t-stack and t-stack-s390 for s390*-*-linux.
* config/s390/morestack.S: New file.
* config/s390/t-stack-s390: New file
/testsuite/gcc.target/s390/morestack.c
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 8f528b2..26d600f 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,7 @@
+2016-02-05 Marcin Kościelnicki <koria...@0x04.net>:
+
+ * gcc.targe
-msmall-exec, it doesn't really matter
much either.)
Ah right, I've added PC16DBL to the "always call" list.
Bye,
Ulrich
I've updated and resubmitted the gold patch.
Marcin Kościelnicki
On 03/02/16 18:27, Ulrich Weigand wrote:
Marcin Kościelnicki wrote:
libgcc/ChangeLog:
* config.host: Use t-stack and t-stack-s390 for s390*-*-linux.
* config/s390/morestack.S: New file.
* config/s390/t-stack-s390: New file.
* generic-morestack.c
bjects using split stack.
Bye,
Ulrich
Fair enough. Here's what I'm going to implement in gold:
- any PLT relocation: call
- PC32DBL on a larl: non-call
- PC32DBL otherwise: call
- any other relocation: non-call
Does that sound right?
Marcin Kościelnicki
@@
+2016-02-03 Marcin Kościelnicki <koria...@0x04.net>
+
+ * common/config/s390/s390-common.c (s390_supports_split_stack):
+ New function.
+ (TARGET_SUPPORTS_SPLIT_STACK): New macro.
+ * config/s390/s390-protos.h: Add s390_expand_split_stack_prologue.
+ * config/s39
The second issue I'm still not sure about is the magic nop marker
for frameless functions. In an earlier mail you wrote:
Both currently supported
architectures always emit split-stack code on every function.
At least for rs6000 this doesn't appear to be true; in
On 02/02/16 19:33, Ulrich Weigand wrote:
Marcin Kościelnicki wrote:
Here we go. I've also removed the "see below", since I don't really
see anything below...
The "see below" refers to this code (which I agree isn't really obvious):
if (TARGET_TPF_PROFILING)
{
/* Generate a
eLog b/gcc/ChangeLog
index 9a2cec8..568dff4 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,33 @@
+2016-02-02 Marcin Kościelnicki <koria...@0x04.net>
+
+ * common/config/s390/s390-common.c (s390_supports_split_stack):
+ New function.
+ (TARGET_SUPPORTS_SPLIT_S
onfig/s390/morestack.S
create mode 100644 libgcc/config/s390/t-stack-s390
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 9a2cec8..af86079 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,40 @@
+2016-02-02 Marcin Kościelnicki <koria...@0x04.net>
+
+ * common/config/s3
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 9a2cec8..af86079 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,40 @@
+2016-02-02 Marcin Kościelnicki <koria...@0x04.net>
+
+ * common/config/s390/s390-common.c (s390_supports_split_stack):
+ New fu
,
-Andreas-
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index c881d52..71f6f38 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,5 +1,38 @@
2016-01-16 Marcin Kościelnicki <koria...@0x04.net>
+ * common/config/s390/s390-common.c (s390_supports_split_stack):
+ New fu
On 22/01/16 08:44, Andreas Krebbel wrote:
On 01/22/2016 12:10 AM, Jeff Law wrote:
On 01/21/2016 03:05 AM, Andreas Krebbel wrote:
On 01/02/2016 08:16 PM, Marcin Kościelnicki wrote:
When an unconditional jump with side effects targets an immediately
following label, rtl_tidy_fallthru_edge
On 21/01/16 11:12, Andreas Krebbel wrote:
On 01/15/2016 10:08 PM, Marcin Kościelnicki wrote:
On 15/01/16 19:38, Andreas Krebbel wrote:
Marcin,
your implementation looks very good to me. Thanks!
But please be aware that we deprecated the support of g5 and g6 and intend to
remove that code
-- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2016-01-21 Marcin Kościelnicki <koria...@0x04.net>
+
+ * config/s390/s390.c (s390_function_profiler): Add a new sequence
+ for z900+ CPUs in 31-bit mode.
+
2016-01-21 Richard Biener <rguent...@suse.de>
* gra
On 21/01/16 11:05, Andreas Krebbel wrote:
On 01/02/2016 08:16 PM, Marcin Kościelnicki wrote:
When an unconditional jump with side effects targets an immediately
following label, rtl_tidy_fallthru_edge is called. Since it has side
effects, it doesn't remove the jump, but the label is still
On 21/01/16 10:58, Andreas Krebbel wrote:
On 01/20/2016 02:16 PM, Andreas Krebbel wrote:
On 01/02/2016 08:16 PM, Marcin Kościelnicki wrote:
It seems at some point the .size hook was hijacked to emit some
machine-specific directives, and the actual .size directive was
forgotten. This caused
On 20/01/16 14:11, Andreas Krebbel wrote:
On 01/02/2016 08:16 PM, Marcin Kościelnicki wrote:
Previously, .rodata was hardcoded. For C++ vague linkage functions,
this resulted in needlessly duplicated literals. With the new split
stack support, this resulted in link errors, due to .rodata
(-)
create mode 100644 libgcc/config/s390/morestack.S
create mode 100644 libgcc/config/s390/t-stack-s390
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index c881d52..71f6f38 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,5 +1,38 @@
2016-01-16 Marcin Kościelnicki <koria...@0x04.
don't need #3
anymore once g5/g6 support is removed, but #4 might still be necessary -
we still have that unconditional jump.
Marcin Kościelnicki
On 03/01/16 04:20, Ian Lance Taylor wrote:
On Sat, Jan 2, 2016 at 11:16 AM, Marcin Kościelnicki <koria...@0x04.net> wrote:
The differences start in the __morestack calling convention. Basically,
since pushing things on stuck is unwieldy and there's only one free
register (%r0 could b
On 03/01/16 04:20, Ian Lance Taylor wrote:
On Sat, Jan 2, 2016 at 11:16 AM, Marcin Kościelnicki <koria...@0x04.net> wrote:
The differences start in the __morestack calling convention. Basically,
since pushing things on stuck is unwieldy and there's only one free
register (%r0 could b
/ChangeLog b/gcc/ChangeLog
index 4c7046f..a4f4dff 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,5 +1,46 @@
2016-01-02 Marcin Kościelnicki <koria...@0x04.net>
+ * common/config/s390/s390-common.c (s390_supports_split_stack):
+ New function.
+ (TARGET_SUPPORTS_SPLIT
insertions(+), 2 deletions(-)
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 23ce209..2c572a7 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,9 @@
+2016-01-02 Marcin Kościelnicki <koria...@0x04.net>
+
+ * config/s390/s390.md (pool_section_start): Use switch_to_s
/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,5 +1,10 @@
2016-01-02 Marcin Kościelnicki <koria...@0x04.net>
+ * config/s390/s390.c (s390_asm_declare_function_size): Add code
+ to actually emit the .size directive.
+
+2016-01-02 Marcin Kościelnicki <koria...@0x04.net>
+
*
Here's my attempt at adding -fsplit-stack support for s390 targets
(bug 68191). Patches 1 and 2 fix s390-specific issues affecting split
stack code, and can be pushed independently of the main course. Patches
3 and 4 attempt to fix target-independent issues involving unconditional
jumps with
/ChangeLog
@@ -1,5 +1,11 @@
2016-01-02 Marcin Kościelnicki <koria...@0x04.net>
+ * function.c (reposition_prologue_and_epilogue_notes): Avoid
+ verification error if the last insn of prologue is an unconditional
+ jump.
+
+2016-01-02 Marcin Kościelnicki <koria...
/gcc/ChangeLog
@@ -1,5 +1,10 @@
2016-01-02 Marcin Kościelnicki <koria...@0x04.net>
+ * cfgrtl.c (rtl_tidy_fallthru_edge): Bail for unconditional jumps
+ with side effects.
+
+2016-01-02 Marcin Kościelnicki <koria...@0x04.net>
+
35 matches
Mail list logo