RFR (XS) 8079682: [mlvm] remove SAM code in MHTransformationGen

[JC Beyler]

Hi Igor,

Sounds good to me, here is the version with the metadata so that it's in
place:
http://cr.openjdk.java.net/~jcbeyler/8079682/webrev.01/

Thanks again!
Jc

On Mon, Aug 6, 2018 at 1:56 PM Igor Ignatyev <igor.ignatyev at oracle.com>
wrote:

> JC,
>
> as this fix is a trivial clean up, you don't need a second reviewer. I'll
> push your patch today.
>
> -- Igor
>
> On Aug 6, 2018, at 11:41 AM, JC Beyler <jcbeyler at google.com> wrote:
>
> Thanks Igor,
>
> Would anyone else on the list care to give me a review for this small
> webrev?
>
> Thanks in advance!
> Jc
>
> On Fri, Aug 3, 2018 at 2:44 PM Igor Ignatyev <igor.ignatyev at oracle.com>
> wrote:
>
>> Hi Jc,
>>
>> looks good to me.
>>
>> -- Igor
>>
>> On Aug 3, 2018, at 2:32 PM, JC Beyler <jcbeyler at google.com> wrote:
>>
>> Hi all,
>>
>> I have a small webrev test fix for a MLVM test:
>>
>> Webrev: http://cr.openjdk.java.net/~jcbeyler/8079682/webrev.00/
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8079682
>>
>> Basically, I removed the case 9 of the switch, reduced the integer
>> generation by 1 and shifted the cases 10 and 11.
>>
>> I removed the case 9 and the MHSamTF object and class was not used
>> anywhere else so I removed that as well (as per the bug).
>>
>> Let me know what you think,
>> Jc
>>
>>
>>
>
> --
>
> Thanks,
> Jc
>
>
>

-- 

Thanks,
Jc
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20180806/5010b86f/attachment.html>