On 07/04/2020 16:33, Stefan Liebler wrote:
> On 4/1/20 6:20 PM, Stefan Liebler wrote:
>> On 4/1/20 12:50 PM, Iain Buclaw wrote:
>>>
>>>
>>> On 01/04/2020 08:28, Stefan Liebler wrote:
>>>> ping
>>>>
>>>
>>> Hi Stefan,
>>>
>>> As I've already said, I think that the name should be __ibmz_get_tls_offset
>>> to make clear that it is an internal function.
>>>
>>> Other than that, looks good to me.
>>>
>>> Iain.
>>>
>>
>> Hi Iain,
>>
>> Sorry. I've missed your comment in the bugzilla.
>> I've updated the name to __ibmz_get_this_offset.
>> Nothing else is changed in the attached patch.
>>
>> Please commit the patch upstream.
>> Do you also close the bugzilla as soon as committed?
>>
>> Regarding the mentioned musl-patch in your bugzilla comment:
>> Yes, the diff looks like not conflicting.
>>
>> Thanks,
>> Stefan
>
> Hi Iain,
>
> Andreas has just committed the other patch "S/390: Fix layout of struct
> sigaction_t" to gcc after your pull-request was merged
> (https://github.com/dlang/druntime/pull/3020).
>
> To me it seems that this patch is not simply pull-request-able to
> https://github.com/dlang/druntime.
Correct, all sources touched are local to gcc only.
> As you've already mentioned "Other than that, looks good to me.", is this gcc
> patch okay to commit from your side? Then Andreas can also commit it and we
> can close the bugzilla.
>
No problems with it being committed.
Regards
Iain.