On Wed, 26 Apr 2023 18:35:45 GMT, Thomas Stuefe <stu...@openjdk.org> wrote:

>> src/hotspot/cpu/arm/sharedRuntime_arm.cpp line 649:
>> 
>>> 647: 
>>> 648:   __ flush();
>>> 649:   return AdapterHandlerLibrary::new_entry(fingerprint, i2c_entry, 
>>> c2i_entry, c2i_unverified_entry);
>> 
>> This change seems out of place... what's the story here?
>
> This is a local revert of *8303154: Investigate and improve instruction cache 
> flushing during compilation* - the missing flush caused random crashes, but I 
> did not have time to investigate. I reverted the flush, crashes were gone.
> 
> If needed I may revisit this when there is time.

I'll remove it. Tests have to wait on arm for 
https://bugs.openjdk.org/browse/JDK-8305387 though.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/10907#discussion_r1180108118

Reply via email to