On 13/11/2018 14:00, Vicente Romero wrote:
any other comment after the last iteration? we are in a bit of a hurry to push this before the JDK 12 train departs :(
The original patch updated all the use sites (and tests) to specify ASM7 for the API version. I just checked the webrev again now and it seems to be just the ASM refresh now. Assuming all the tests are passing and you've sorted out the mvlm test issues with Igor then I suggest go ahead with this push and we can update the sites, as needed, at a later time.

-Alan

Reply via email to