On Fri, 3 Feb 2023 17:56:45 GMT, Maurizio Cimadamore <mcimadam...@openjdk.org> wrote:
>> Adam Sotona has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Classfile API moved under jdk.internal.classfile package > > src/java.base/share/classes/jdk/internal/classfile/Attributes.java line 774: > >> 772: */ >> 773: public static AttributeMapper<?> standardAttribute(Utf8Entry name) { >> 774: int hash = name.hashCode(); > > If we have a map, why do we need this "inlined" map here? Performance reasons? Yes, performance is the main reason. I'll note to do a fresh differential performance benchmarks with a HashMap. ------------- PR: https://git.openjdk.org/jdk/pull/10982