Re: RFR: 8349753: Incorrect use of CodeBlob::is_buffer_blob() in few places

2025-02-13 Thread Vladimir Kozlov
On Thu, 13 Feb 2025 01:22:55 GMT, Vladimir Kozlov wrote: > `CodeBlob::is_buffer_blob()` method is incorrectly used in few places because > BufferBlob is not "leaf" class. You need to add checks for its subclasses too. > > I also updated statistic output for CodeCache (`-XX:+PrintCodeCache > -X

Re: RFR: 8349753: Incorrect use of CodeBlob::is_buffer_blob() in few places

2025-02-13 Thread Aleksey Shipilev
On Thu, 13 Feb 2025 01:22:55 GMT, Vladimir Kozlov wrote: > `CodeBlob::is_buffer_blob()` method is incorrectly used in few places because > BufferBlob is not "leaf" class. You need to add checks for its subclasses too. > > I also updated statistic output for CodeCache (`-XX:+PrintCodeCache > -X

Re: RFR: 8349753: Incorrect use of CodeBlob::is_buffer_blob() in few places

2025-02-12 Thread Vladimir Kozlov
On Thu, 13 Feb 2025 01:22:55 GMT, Vladimir Kozlov wrote: > `CodeBlob::is_buffer_blob()` method is incorrectly used in few places because > BufferBlob is not "leaf" class. You need to add checks for its subclasses too. > > I also updated statistic output for CodeCache (`-XX:+PrintCodeCache > -X

Re: RFR: 8349753: Incorrect use of CodeBlob::is_buffer_blob() in few places

2025-02-12 Thread Dean Long
On Thu, 13 Feb 2025 01:22:55 GMT, Vladimir Kozlov wrote: > `CodeBlob::is_buffer_blob()` method is incorrectly used in few places because > BufferBlob is not "leaf" class. You need to add checks for its subclasses too. > > I also updated statistic output for CodeCache (`-XX:+PrintCodeCache > -X

RFR: 8349753: Incorrect use of CodeBlob::is_buffer_blob() in few places

2025-02-12 Thread Vladimir Kozlov
`CodeBlob::is_buffer_blob()` method is incorrectly used in few places because BufferBlob is not "leaf" class. You need to add checks for its subclasses too. I also updated statistic output for CodeCache (`-XX:+PrintCodeCache -XX:+Verbose`) and corresponding test to reflect current state of code