Request for review (M): 7029017 Additional architecture support for c2 compiler

[John Rose]

On Mar 21, 2011, at 10:37 AM, Roland Westrelin wrote:

> Hi Vladimir,
> 
>> Did you consider to use Matcher property instead of new flag and new ideal node
>> code? See how we do Matcher::clone_shift_expressions.
> 
> I would like the extra and operations to be optimized out when possible. That won't happen if I do the work at the matcher level, right?

See if you can fold the And node into the Shift node in the matcher, like the matcher folds the shift and add into the addressing mode.

-- John