On Fri, 31 Mar 2023 18:38:43 GMT, Xin Liu <x...@openjdk.org> wrote: >> I see, you use it in escape.cpp. Okay. I need to review changes there too. > > or you could construct a temporary PhaseMacroExpand object in EA. > > I see that you convert many member function to static so you can query in EA. > the only blocker is _igvn.
That seems a good idea. Together with some other refactoring I decided to revert making the methods static and instead use them through an instance of ME. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/12897#discussion_r1158698606