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 tracep
suite/ChangeLog
@@ -1,3 +1,14 @@
+2016-02-19 Marcin Kościelnicki
+
+ * 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-unavailable.exp (proc_trace_una
b/gdb/testsuite/ChangeLog
@@ -1,5 +1,9 @@
2016-02-19 Marcin Kościelnicki
+ * gdb.trace/ftrace.exp: Remove unnecessary target check.
+
+2016-02-19 Marcin Kościelnicki
+
* gdb.trace/entry-values.exp: Surround $call_insn with '\y'.
* lib/trace-support.exp: Change x
db/testsuite/ChangeLog
+++ b/gdb/testsuite/ChangeLog
@@ -1,5 +1,10 @@
2016-02-19 Marcin Kościelnicki
+ * 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
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
geLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,33 @@
+2016-02-14 Marcin Kościelnicki
+
+ * 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_pro
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
4 gcc/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 :
+
+ * gcc.target/s390/morestack.c: New
ut since basically
nobody ever uses -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
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
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 (__splitstack_f
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
rs6000_expand_split_stack_p
-1,3 +1,33 @@
+2016-02-03 Marcin Kościelnicki
+
+ * 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/s390/s390
/gcc/ChangeLog b/gcc/ChangeLog
index 9a2cec8..568dff4 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,33 @@
+2016-02-02 Marcin Kościelnicki
+
+ * common/config/s390/s390-common.c (s390_supports_split_stack):
+ New function.
+ (TARGET_SUPPORTS_SPLIT_STACK): New macro.
+
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 B
100644 libgcc/config/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
+
+ * common/config/s390/s390-common.c (
ibgcc/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
+
+ * common/config/s390/s390-common.c (s390_supports_split_stack):
+ New function.
+
ts inline.
Bye,
-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
+ * common/config/s390/s390-common.c (s390_supports_split_stack):
+
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 is
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
+
+ * config/s390/s390.c (s390_function_profiler): Add a new sequence
+ for z900+ CPUs in 31-bit mode.
+
2016-01-21 Richard Biener
* graphite-optimize-isl.c (get_schedule_map): Fix typo.
di
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
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
+), 6 deletions(-)
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
+ * com
out getting #3 and #4 reviewed. We 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 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 be used for static chain
On 03/01/16 04:20, Ian Lance Taylor wrote:
On Sat, Jan 2, 2016 at 11:16 AM, Marcin Kościelnicki 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 be used for static chain
iff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 4c7046f..a4f4dff 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,5 +1,46 @@
2016-01-02 Marcin Kościelnicki
+ * common/config/s390/s390-common.c (s390_supports_split_stack):
+ New function.
+ (TARGET_SUPPORTS_SPLIT_ST
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
+
+ * config/s390/s390.md (pool_section_start): Use switch_to_section
+ to select proper read
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 side
/ChangeLog
@@ -1,5 +1,11 @@
2016-01-02 Marcin Kościelnicki
+ * 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
+
* config/s390/s390.c
a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,5 +1,10 @@
2016-01-02 Marcin Kościelnicki
+ * config/s390/s390.c (s390_asm_declare_function_size): Add code
+ to actually emit the .size directive.
+
+2016-01-02 Marcin Kościelnicki
+
* config/s390/s390.md (pool_section_start)
+++ b/gcc/ChangeLog
@@ -1,5 +1,10 @@
2016-01-02 Marcin Kościelnicki
+ * cfgrtl.c (rtl_tidy_fallthru_edge): Bail for unconditional jumps
+ with side effects.
+
+2016-01-02 Marcin Kościelnicki
+
* function.c (reposition_prologue_and_epilogue_notes): Avoid
verific
35 matches
Mail list logo
