Request for review (L): 6797305: Add LoadUB and LoadUI opcode class
Christian Thalinger
Christian.Thalinger at Sun.COM
Fri Feb 27 08:15:25 PST 2009
On Thu, 2009-02-26 at 09:02 -0800, Tom Rodriguez wrote:
> As far as sparc goes I wouldn't mind if they were changed to use the
> MacroAssembler too. Someday we'd like to convert all the inc_encodes
> to use MacroAssembler.
Hmm, I don't know how to do it properly on SPARC. There are only two
(three) versions of emit functions:
ldsb( Register s1, Register s2, Register d)
ldsb( Register s1, int simm13a, Register d)
(And this one:
ldsb( const Address& a, Register d, int offset )
)
I tried something like that:
ins_encode %{
__ ldsb($mem$$base$$Register, $mem$$disp, $dst$$Register);
%}
but that does work (I know the $$Register is wrong). The question is:
would it be correct to use $mem$$base and $mem$$disp and assume $mem
$$index and $mem$$scale are zero or not used?
-- Christian
More information about the hotspot-compiler-dev
mailing list