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 from
the back-end with the next GCC version. So I would prefer if you could remove
all the
!TARGET_CPU_ZARCH stuff from the implementation and just error out if
split-stack is enabled with
-march g5/g6. It currently makes the implementation more complicated and would
have to be removed
anyway in the future.
Thanks!
https://gcc.gnu.org/ml/gcc-patches/2015-12/msg01854.html
Bye,
-Andreas-
Very well, I'll do that.
Btw, as for dropping support for g5/g6: I've noticed
s390_function_profiler could also use larl+brasl for -m31 given
TARGET_CPU_ZARCH. Should I submit a patch for that? I'm asking because
gold with -fsplit-stack needs to know the exact sequence used, so if
it's going to change after g5/g6 removal, I'd better add it to gold now
(and make gcc always emit it for non-g5/g6, so that gold won't need to
look at the old one).
Yes please, that would be great. Good catch!
Thanks!
-Andreas-
I've submitted the gcc patch, and will soon update the gold patch.
Marcin Kościelnicki
