LGTM, but I noticed similar code on x64. Apparently, it is possible (yet unlikely) to have an x64 chip with no SSE3 support, so you need to fix that too.
Maybe while you're at it, you should double check that CpuFeatures::IsSupported(CpuFeatures::SSE3) returns what you expect on x64 platforms? http://codereview.chromium.org/172086 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
