RFR: 8241042: x86_64: Improve Assembler generation

Claes Redestad claes.redestad at oracle.com
Tue Mar 17 10:55:08 UTC 2020


On 2020-03-17 00:56, Vladimir Kozlov wrote:
> Something messed up. Move prefixq(Address adr, Register src) near 
> corresponding get_prefixq().
> Currently an other method Assembler::prefix(Address adr, Register reg, 
> bool byteinst) mixed up in changes.
> Which brings question: why you did not do the same for prefix() methods?
> 
> May be you can move get_prefixq() changes into separate RFE so we can 
> continue discussion?

Ok, let's tease these apart.

> 
> Changes emit_int* can be pushed first.

Here's a version with only the emit_intxx changes:

http://cr.openjdk.java.net/~redestad/8241042/open.03/

I'll file an RFE and keep polishing the prefixq improvements there.

/Claes


More information about the hotspot-compiler-dev mailing list