6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-07-26 05:25:47 +00:00
Code Issues Projects Releases Wiki Activity

Fix a bug found by inspection; in the __GNUC__ code, the alignment

Browse Source 
doesn't apply to the type, only to the variable, so subsequent uses
of U which expect it to be aligned weren't actually aligned.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@98843 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Dan Gohman
2010-03-18 18:40:47 +00:00
parent 8bba4dd9bc
commit 74cdc75313
1 changed files with 9 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
include/llvm/ADT/SmallVector.h
View File

@@ -57,17 +57,18 @@ protected:
// something else. An array of char would work great, but might not be // something else. An array of char would work great, but might not be
// aligned sufficiently. Instead, we either use GCC extensions, or some // aligned sufficiently. Instead, we either use GCC extensions, or some
// number of union instances for the space, which guarantee maximal alignment. // number of union instances for the space, which guarantee maximal alignment.
struct U {
#ifdef __GNUC__ #ifdef __GNUC__
typedef char U; char X __attribute__((aligned));
U FirstEl __attribute__((aligned));
#else #else
union U { union U {
double D; double D;
long double LD; long double LD;
long long L; long long L;
void *P; void *P;
} FirstEl; } X;
#endif #endif
} FirstEl;
// Space after 'FirstEl' is clobbered, do not add any instance vars after it. // Space after 'FirstEl' is clobbered, do not add any instance vars after it.
protected: protected: