On Tue, 16 Mar 2021 07:29:34 GMT, Vyom Tewari <vtew...@openjdk.org> wrote:
>> Jaikiran Pai has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Use Chris' suggested solution of avoiding deadlock > > Looks good to me. Holder pattern is easier to understand, I agree. Perhaps you could just add a warning to the DynamicConstantDesc.ofCanonical() method javadoc/comment about what *NOT* to do in order to not fall into the deadlock trap again... (like: don't call this method from static initializer of ConstantDescs or such). Otherwise it looks good to me to. ------------- PR: https://git.openjdk.java.net/jdk/pull/2893