commit 486cc19e1e2eca4231f760117e95090c03e2d8c6
author Nicolas Geoffray <ngeoffray@google.com> Mon Dec 08 18:00:55 2014 +0000
committer Nicolas Geoffray <ngeoffray@google.com> Tue Dec 09 10:18:04 2014 +0000
tree d9d21691ea0e019c0940d6c72b8593c75d810662
parent 6c964c98400b8c0949d5e369968da2d4809b772f

Explicitly mask constants in shift operations.

The assemblers expect an int8, so we mask ahead of calling them.

Change-Id: Id668cda6853fa365ac02531bf7aae288cad20fcd

compiler/optimizing/code_generator_x86.cc

diff --git a/compiler/optimizing/code_generator_x86.cc b/compiler/optimizing/code_generator_x86.cc
index 8a0c2de..4757235 100644
--- a/compiler/optimizing/code_generator_x86.cc
+++ b/compiler/optimizing/code_generator_x86.cc
@@ -2391,7 +2391,7 @@
           __ shrl(first_reg, second_reg);
         }
       } else {
-        Immediate imm(second.GetConstant()->AsIntConstant()->GetValue());
+        Immediate imm(second.GetConstant()->AsIntConstant()->GetValue() & kMaxIntShiftValue);
         if (op->IsShl()) {
           __ shll(first_reg, imm);
         } else if (op->IsShr()) {