Hi Gustavo, thanks for your improvements. You can also remove the trailing \t from the opto assembly. Note that there's no need to re-push newer version to jdk/submit when only PPC files were changed. jdk/submit doesn't look at them.
Best regards, Martin -----Original Message----- From: Gustavo Romero <[email protected]> Sent: Mittwoch, 12. Dezember 2018 03:08 To: Michihiro Horie <[email protected]> Cc: [email protected]; [email protected]; Doerr, Martin <[email protected]>; Roger Riggs <[email protected]>; Vladimir Kozlov <[email protected]>; Simonis, Volker <[email protected]> Subject: Re: RFR: 8213754: PPC64: Add Intrinsics for isDigit/isLowerCase/isUpperCase/isWhitespace Hi Michi, On 12/11/2018 11:12 PM, Michihiro Horie wrote: > Thank you for finding the issue on Power8. You do not need a check with > has_darn in the ppc.ad. It is better to add a check in vm_versoin_ppc. I agree. > I uploaded webrev.08 based on your webrev.07. (Thanks for the enhancement of > opto assembly and removing trailing spaces!) > http://cr.openjdk.java.net/~mhorie/8213754/webrev.08/ > <http://cr.openjdk.java.net/%7Emhorie/8213754/webrev.08/> Thanks for the updated webrev. Looks good! I've just pushed webrev.08 to jdk/submit expecting no failures as .07 passed fine. Once I get the jdk/submit results tomorrow I'll push. Best regards, Gustavo
