[PATCH] [RFC] [GOLD] s390 -fsplit-stack support.

Cary Coutant ccoutant@gmail.com
Sun Dec 13 22:42:00 GMT 2015


> Thanks for the patches!  I've updated mine, and the changes are now limitted
> to s390-specific code.  The output_view patch has a minor problem: it
> returns a const unsigned char *, but I need a read-write view (I have to
> bump the frame size in the parameter block).  I've just const_casted around
> it for now, but I suppose we should change the return type instead?

Oops, sorry! I've committed the attached patch to remove the const.

2015-12-13  Cary Coutant  <ccoutant@gmail.com>

        * object.h (Object::get_output_view): remove const from return type.
        (Object::do_get_output_view): Likewise.
        (Sized_relobj_file::do_get_output_view): Likewise.
        * reloc.cc (Sized_relobj_file::do_get_output_view): Likewise.

> Also, I still have an icky static_cast in the "find parameter block"
> sequence. but I suppose it's here to stay:
>
> +             Sized_relobj_file<size, true> *object_sized =
> +               static_cast<Sized_relobj_file<size, true> *>(object);

Yes, we have a few other similar casts. It's not a high priority, but
I'll eventually see about adding a virtual function to the Object
class to return a Sized_relobj_file pointer to itself.

> Btw, this patch shouldn't be landing yet, I still have to finish the gcc
> side.

No problem; let me know when it's ready.

-cary



More information about the Binutils mailing list