On Fri, 8 Jan 2021 11:55:21 GMT, Kim Barrett <kbarr...@openjdk.org> wrote:
> [Can't comment on this inline.] I'd prefer DUIterator_Last::operator= be > changed to =default, for consistency with the copy constructor. That would > require fixing the return type too. Thanks for your comment. Agree. Will update the code. > src/hotspot/share/opto/node.hpp line 1461: > >> 1459: // initialize to garbage >> 1460: >> 1461: DUIterator_Last (const DUIterator_Last& that) = default; > > Nit - remove space before parameter list. Thanks for point it out. Will remove. ------------- PR: https://git.openjdk.java.net/jdk/pull/1874
