[newlib-cygwin] Cygwin: cygtls: drop useless alignment directives

Corinna Vinschen corinna@sourceware.org
Mon Sep 5 17:52:43 GMT 2022


https://sourceware.org/git/gitweb.cgi?p=newlib-cygwin.git;h=783133b753d8c62077a575a9b1adb3f334615d02

commit 783133b753d8c62077a575a9b1adb3f334615d02
Author: Corinna Vinschen <corinna@vinschen.de>
Date:   Mon Sep 5 19:49:11 2022 +0200

    Cygwin: cygtls: drop useless alignment directives
    
    The patch fixing the alignment of _cygtls::context accidentally
    pushed the desperate attempt to automate the alignment by using
    another, non-working variation of attribute((aligned)).  Drop it.
    
    Fixes: dcab768cb93e ("Cygwin: cygtls: fix context alignment")
    Signed-off-by: Corinna Vinschen <corinna@vinschen.de>

Diff:
---
 winsup/cygwin/local_includes/cygtls.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/winsup/cygwin/local_includes/cygtls.h b/winsup/cygwin/local_includes/cygtls.h
index 61c4cbeef..f67e9136c 100644
--- a/winsup/cygwin/local_includes/cygtls.h
+++ b/winsup/cygwin/local_includes/cygtls.h
@@ -185,7 +185,7 @@ public: /* Do NOT remove this public: line, it's a marker for gentls_offsets. */
      If you prepend cygtls members here, make sure context stays 16 byte
      aligned. The gentls_offsets script checks for that now and fails
      if the alignment is wrong. */
-  ucontext_t __attribute__((__aligned__(__alignof__(long double)))) context;
+  ucontext_t context;
   DWORD thread_id;
   siginfo_t infodata;
   struct pthread *tid;


More information about the Cygwin-cvs mailing list