Skip to content

Commit

Permalink
Fix ABI issue: Longs really do need to be only 4 byte aligned on X86.
Browse files Browse the repository at this point in the history
This bug caused miscompilation of programs using 'struct stat', but only if
compiled with support for 64-bit filesystems.  This could in theory effect
other things, but only if the LLVM code shared data structures with native code.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@7928 91177308-0d34-0410-b5e6-96231b3b80d8
lattner committed Aug 17, 2003
1 parent 57e5a70 commit 7540565
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion lib/Target/X86/X86TargetMachine.cpp
Original file line number Diff line number Diff line change
@@ -39,7 +39,8 @@ X86TargetMachine::X86TargetMachine(unsigned Config)
(Config & TM::EndianMask) == TM::LittleEndian,
(Config & TM::PtrSizeMask) == TM::PtrSize64 ? 8 : 4,
(Config & TM::PtrSizeMask) == TM::PtrSize64 ? 8 : 4,
(Config & TM::PtrSizeMask) == TM::PtrSize64 ? 8 : 4),
(Config & TM::PtrSizeMask) == TM::PtrSize64 ? 8 : 4,
4, (Config & TM::PtrSizeMask) == TM::PtrSize64 ? 8 : 4),
FrameInfo(TargetFrameInfo::StackGrowsDown, 8/*16 for SSE*/, 4) {
}

0 comments on commit 7540565

Please sign in to comment.