Request for reviews (M): 7201026: add vector for shift count

Vladimir Kozlov vladimir.kozlov at
Mon Oct 1 15:13:07 PDT 2012

We already have it, adlparse.cpp:

     else if      (!strcmp(ident, "match")) {
       // Allow one instruction have several match rules.

It looks like I added it 6 years ago :)

I tried with this changes and it works :O

I will update webrev and resend it after some testing.


Christian Thalinger wrote:
> On Oct 1, 2012, at 5:20 AM, Roland Westrelin <roland.westrelin at> wrote:
>> It looks good.
>> Why do we need a different node type for left and right shifts? Wouldn't a single one be sufficient?
> Where is the multiple-match-rules-per-instruct RFE?  -- Chris
>> Roland.

More information about the hotspot-compiler-dev mailing list