On Fri, 24 Mar 2023 23:37:29 GMT, Vladimir Kozlov <k...@openjdk.org> wrote:

>> I had to make this method static because it uses `value_from_mem` - which I 
>> also made static. I had to make `value_from_mem` static so that I can use it 
>> outside PhaseMacroExpand.
>
> 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.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/12897#discussion_r1154785040

Reply via email to