Maybe noreg-trivial is for lazy people. I'll remove it and add a comment. Thanks Max
> On Oct 22, 2018, at 11:30 PM, Sean Mullan <sean.mul...@oracle.com> wrote: > > Looks fine to me. It is unusual to have two noreg labels on a bug. Is that > acceptable? I would probably choose one or the other. noreg-hard seems more > appropriate to me. > > --Sean > > On 10/15/18 9:34 PM, Weijun Wang wrote: >> Please take a review at >> http://cr.openjdk.java.net/~weijun/8212216/webrev.00/ >> This bug is reported and fixed by Nico Williams <n...@twosigma.com>. >> No regression test, memory leak. >> Thanks >> Max