David Holmes

Okay so we currently have `parse_integer` that calls `parse_integer_impl` to do the raw numeric parsing, then applies the modifiers conversions itself. I don't see why that structure needs to change...

Okay I agree it makes no sense to have `os::signal` as shared code because you can't use it in a platform independent manner. I can take that new bug (thanks...

@shipilev this has broken our linux aarch64 builds! [2022-10-20T07:26:59,542Z] workspace/open/src/hotspot/cpu/aarch64/assembler_aarch64.cpp: In member function 'void Address::lea(MacroAssembler*, Register) const': [2022-10-20T07:26:59,542Z] workspace/open/src/hotspot/cpu/aarch64/assembler_aarch64.cpp:138:5: error: this 'else' clause does not guard... [-Werror=misleading-indentation] [2022-10-20T07:26:59,542Z] 138 |...

This appears to be a 10 year old bug in gcc. Have we ever had any issues reported because of this? Inserting a workaround now seems rather late. Any FP-using...

Correction: I was under the false impression the FP-state was shared at the process level but I've been told it is per-thread. So the scope of the problem is much...

For the benefit of the mailing list my previous comment has been edited/corrected: Correction: I was under the false impression the FP-state was shared at the process level but I've...

/label add hotspot-jfr

@mgronlun my request is that this filtering be done inside the commit logic by the JFR code, not at the site where the event is generated - ie this internal-jfr-event...

> I need to see if there is some historical info on the initial implementation here. Nope. Lets see if anyone else knows why the code may have originally been...

> We're not seeing anyone wanting to chime in on this No, unfortunately the committers who have been very active in this code most recently have not commented on it.