Reviewers: Christian Plesner Hansen, Description: 8-byte align C stack on Linux and Windows.
Please review this at http://codereview.chromium.org/42116 SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/ Affected files: M src/platform-linux.cc M src/platform-win32.cc Index: src/platform-win32.cc =================================================================== --- src/platform-win32.cc (revision 1495) +++ src/platform-win32.cc (working copy) @@ -1298,8 +1298,8 @@ int OS::ActivationFrameAlignment() { - // No constraint on Windows. - return 0; + // Floating point code runs faster if the stack is 8-byte aligned. + return 8; } Index: src/platform-linux.cc =================================================================== --- src/platform-linux.cc (revision 1495) +++ src/platform-linux.cc (working copy) @@ -212,8 +212,8 @@ int OS::ActivationFrameAlignment() { - // No constraint on Linux. - return 0; + // Floating point code runs faster if the stack is 8-byte aligned. + return 8; } --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
