Request for review (L): 6797305: Add LoadUB and LoadUI opcode class

[Tom Rodriguez]

What does the failure output look like?

tom

On Mar 6, 2009, at 7:49 AM, Christian Thalinger wrote:

> On Fri, 2009-03-06 at 12:15 +0100, Christian Thalinger wrote:
>> On Fri, 2009-03-06 at 00:45 -0800, Vladimir Kozlov wrote:
>>> Looks good.
>>>
>>> In x86_32.ad in loadB2L you need only 7 bits shift: SAR dst.hi, 7
>>> and in loadS2L only 15 bits.
>>
>> Right.  I will change that and push the changeset.  Thanks Vladimir!
>
> Hmm, JPRT failed and it seems there is a matcher problem.  It matches:
>
>  match(Set dst (ConvI2L (LoadUB mem)));
>
> instead of:
>
>  match(Set dst (ConvI2L src));
>
> But there is no LoadUB node around.  Are there any matcher  
> restrictions
> that I don't know about?
>
> -- Christian
>