On Fri, Sep 9, 2016 at 6:52 AM, Andreas Schwab wrote:
>
> You should use alignof(ucontext_t) instead of hardcoding 16.
Fair enough. Done like so. Bootstrapped and ran Go testsuite on
x86_64-pc-linux-gnu, committed to mainline.
Ian
Index: gcc/go/gofrontend/MERGE
On Sep 09 2016, Ian Lance Taylor wrote:
> Index: libgo/runtime/proc.c
> ===
> --- libgo/runtime/proc.c (revision 239872)
> +++ libgo/runtime/proc.c (working copy)
> @@ -156,6 +156,20 @@ fixcontext(ucontext_t *c)
>
> #end
On Fri, Sep 2, 2016 at 9:27 AM, Andreas Schwab wrote:
> On Sep 02 2016, Ian Lance Taylor wrote:
>
>> On Fri, Sep 2, 2016 at 9:14 AM, Andreas Schwab wrote:
>>>
>>> That breaks libgo on ia64. The problem is that _ucontext_t isn't
>>> properly aligned.
>>
>> Interesting. Thanks for looking into i
On Sep 02 2016, Andreas Schwab wrote:
> On Sep 02 2016, Ian Lance Taylor wrote:
>
>> On Fri, Sep 2, 2016 at 9:14 AM, Andreas Schwab wrote:
>>>
>>> That breaks libgo on ia64. The problem is that _ucontext_t isn't
>>> properly aligned.
>>
>> Interesting. Thanks for looking into it. What is the
On Sep 02 2016, Ian Lance Taylor wrote:
> On Fri, Sep 2, 2016 at 9:14 AM, Andreas Schwab wrote:
>>
>> That breaks libgo on ia64. The problem is that _ucontext_t isn't
>> properly aligned.
>
> Interesting. Thanks for looking into it. What is the required
> alignment? This code should be align
On Fri, Sep 2, 2016 at 9:14 AM, Andreas Schwab wrote:
>
> That breaks libgo on ia64. The problem is that _ucontext_t isn't
> properly aligned.
Interesting. Thanks for looking into it. What is the required
alignment? This code should be aligning it to a pointer boundary.
Ian
That breaks libgo on ia64. The problem is that _ucontext_t isn't
properly aligned.
Andreas.
--
Andreas Schwab, sch...@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5
"And now for something completely different."
