On Fri, 20 Mar 2026 18:03:20 GMT, Coleen Phillimore <[email protected]> wrote:
>> Please review this change to fix GenerateOopMap to not report an unreachable >> bytecode for async exception handling and for the first bytecode in an >> exception catch clause. The first change will generate a safepoint entry >> that doesn't allow async exceptions for bytecodes that cannot trap. The >> second change creates an exception edge for the first bytecode in an >> exception catch clause whether it can trap or not. >> This also adds a GenerateOopMapALot at safeopints to test interpreter oopmap >> generation. Tested JCKs with this option. >> Thanks to @tkrodriguez for the kill003 test. >> Also tested tier1-8. > > Coleen Phillimore has updated the pull request incrementally with five > additional commits since the last revision: > > - Fix copyrights and extraneous diffs > - Add a bit of logging for GenerateOopMapALot. > - Restored special aload, return, monitorexit cases in do_exception_edge. > Make them exceptions from async exception delivery instead. > - Remove monitorenter, monitorexit from do_exception_edge since they can > have exceptions, and turn off GenerateOopMapALot again. > - Generate safepoint checks with and without allowing async exceptions a > different way. Also include the breakpoint code. Dean thank you for your comments and expertise with this change. I cleaned up copyrights and have retested tiers 1-4 (tiers 5-8 in progress). Reopening the PR. ------------- PR Review: https://git.openjdk.org/jdk/pull/30171#pullrequestreview-3983189243
