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
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 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
-- 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: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
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
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):
+
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.
+
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 (
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
/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.
+
-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
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
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
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
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
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
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
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 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
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
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
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
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
+++ 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
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)
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
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
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
+), 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
35 matches
Mail list logo
