Remove alignment from struct XSAVE altogether
authorPavel Labath <pavel@labath.sk>
Fri, 15 Feb 2019 12:20:23 +0000 (12:20 +0000)
committerPavel Labath <pavel@labath.sk>
Fri, 15 Feb 2019 12:20:23 +0000 (12:20 +0000)
I reduced the alignment of this struct in r342029 to avoid compiler
warnings about under-aligned allocations, but it turns out that this
still causes problems with some compilers (see r353778). As I hinted in
r342029, I don't believe any special aligment is necessary here (the
only reason for that would be if we used some aligned SSE instructions to
access this buffer, but I don't see any reason why we should do that),
so here I go all the way, and remove the alignment requirements (except
the ones naturally imposed by basic types) altogether.

llvm-svn: 354125

lldb/source/Plugins/Process/Utility/RegisterContext_x86.h

index 049865c..ad004c8 100644 (file)
@@ -340,7 +340,7 @@ LLVM_PACKED_END
 
 // x86 extensions to FXSAVE (i.e. for AVX and MPX processors)
 LLVM_PACKED_START
-struct LLVM_ALIGNAS(16) XSAVE {
+struct XSAVE {
   FXSAVE i387;      // floating point registers typical in i387_fxsave_struct
   XSAVE_HDR header; // The xsave_hdr_struct can be used to determine if the
                     // following extensions are usable