Re: RFR(7u) 8046233: VerifyError on backward branch

2014-08-27 Thread Karen Kinnear
Harold, Code fix looks good. If you haven't already checked this in - would you mind renaming "has_unique_new_object" to something like "has_nonmatching_uninit_object" or "has_nonmatching_new_object"? If that makes sense to you. thanks, Karen On Aug 22, 2014, at 10:47 AM, harold seigel wrote

Re: RFR(7u) 8046233: VerifyError on backward branch

2014-08-27 Thread harold seigel
Hi Karen, I haven't checked this in. You're the first reviewer. I like "has_nonmatching_new_object". I'll change it to that. Thanks! Harold On 8/27/2014 11:30 AM, Karen Kinnear wrote: Harold, Code fix looks good. If you haven't already checked this in - would you mind renaming "has_uniq

RFR(7u) 8046233: VerifyError on backward branch

2014-08-22 Thread harold seigel
Hi, Please review this small fix for bug 8046233 for JDK-7u. Section 4.10.2.4 of the JVM-7 Spec says: A valid instruction sequence must not have an uninitialized object on the operand stack or in a local vari