On Tue, 23 Mar 2021 13:14:15 GMT, Harold Seigel <hsei...@openjdk.org> wrote:
>> Coleen Phillimore has updated the pull request incrementally with one >> additional commit since the last revision: >> >> missed THREAD that should be CHECK_false argument. > > src/hotspot/share/oops/constantPool.cpp line 1426: > >> 1424: bool match_entry = compare_entry_to(k1, cp2, k2); >> 1425: bool match_operand = compare_operand_to(i1, cp2, i2); >> 1426: return (match_entry && match_operand); > > Is it worth changing this to: return (compare_entry_to(...) && > compare_operand_to(..)); > Then if the first one is false the second call isn't needed? I kind of thought these would make a long complicated expression and so the single use variables is helpful. I don't think performance is important here. ------------- PR: https://git.openjdk.java.net/jdk/pull/3141