Add FOR_INT32_SHIFTS to value-helper.h

Add FOR_INT32_SHIFTS to value-helper.h

R=bmeurer@chromium.org
BUG=

Committed: https://code.google.com/p/v8/source/detail?r=23100

[Benedikt Meurer, 2014-08-13 09:45:02]

https://codereview.chromium.org/470533002/diff/1/test/cctest/compiler/value-h...
File test/cctest/compiler/value-helper.h (right):

https://codereview.chromium.org/470533002/diff/1/test/cctest/compiler/value-h...
test/cctest/compiler/value-helper.h:127: #define FOR_INT32_SHIFTS(var) for
(int32_t var = 0; var < 32; var++)
We don't need that, shift amount is always/automatically unsigned in C++.

[titzer, 2014-08-13 09:57:20]

On 2014/08/13 09:45:02, Benedikt Meurer wrote:
>
https://codereview.chromium.org/470533002/diff/1/test/cctest/compiler/value-h...
> File test/cctest/compiler/value-helper.h (right):
> 
>
https://codereview.chromium.org/470533002/diff/1/test/cctest/compiler/value-h...
> test/cctest/compiler/value-helper.h:127: #define FOR_INT32_SHIFTS(var) for
> (int32_t var = 0; var < 32; var++)
> We don't need that, shift amount is always/automatically unsigned in C++.

In an upcoming CL I am distinguishing more carefully between signed and unsigned
integers and also unifying MachineType and RepType. Having both FOR_INT32_SHIFTS
and FOR_UINT32_SHIFTS is a simpler change now and I think will make that CL
easier when I rebase it.

[Benedikt Meurer, 2014-08-13 09:58:40]

lgtm

[titzer, 2014-08-13 10:30:07]

Committed patchset #1 manually as 23100 (presubmit successful).