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

John Rose john.r.rose at
Mon Mar 21 11:43:07 PDT 2011

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

More information about the hotspot-compiler-dev mailing list