Lutz Schmidt
Lutz Schmidt
>I guess if I backport [openjdk/jdk#8142](https://github.com/openjdk/jdk/pull/8142) then it could be `clean`. But I'm not sure. > You should never consider backporting such a huge, and unrelated, change just to achieve...
> I think we shouldn't allow `MacroAssembler::string_compress(...)` and `MacroAssembler::string_expand(...)` to use vector registers without specifying this effect. That can be solved by adding a KILL effect for all vector registers...
> ... I think that classical benchmarks didn't show a significant performance impact, but I don't remember exactly, either. ... Yes, you need "long" strings (>= 32 characters) for the...
I agree to disable the vector part in the above mentioned emitters. Please do so by enclosing the code in a ``` #if 0 . . . #endif ``` block....
Hi Amit, the function prototype now reads uint MachSpillCopyNode::implementation(C2_MacroAssembler *masm, PhaseRegAlloc *ra_, bool do_size, outputStream *os) const { There is no cbuf argument anymore. Side note: I would prefer to...
> There were several optimization done for this code by @RealLucy [JDK-8223444](https://bugs.openjdk.org/browse/JDK-8223444) and [JDK-8231460](https://bugs.openjdk.org/browse/JDK-8231460). But it is still using `linked list` for free segments. Should we consider something more complex?...
The warning can be understood as a hint to a potential efficiency issue. There is no hard limit for #free blocks. I do not have the Renaissance suite readily runnable...
@bulasevich Thanks for providing the Code Heap Stats. At vm shutdown time, when the stats were taken, there is no pathological state in the code heap. - non-profiled nmethods has...
Never mind. Just read the review comments above saying alternate security providers need to provide all operations in one library. Sorry for the noise. -.-.-.-.-.-.-.-.-.-.-.- While reviewing the backport PR...