On 07.04.20 18:26, Iain Buclaw wrote: > > > 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.
Committed to mainline. Thanks! Andreas > > Regards > Iain. >