Request for reviews (XS): 6879902: CTW failure jdk6_18/hotspot/src/cpu/sparc/vm/assembler_sparc.hpp:845
Christian Thalinger
Christian.Thalinger at Sun.COM
Tue Sep 22 09:57:09 PDT 2009
Tom Rodriguez wrote:
> I actually told him not to since using arg_slot in i2c adapters seemed
> dubious to me since it adds in Interpreter::value_offset_in_bytes().
> I guess since the rest of the i2c code uses it we should use it but
> I'd like an assert that value_offset_in_bytes() == 0 in arg_slot. I
> don't really like that i2c always forces constants into Rdisp instead
> of using the simm13 variants. I think all of these should be using
> RegisterOrConstant and ensure_rs2 to deal with this issue.
Should I roll that into my change? -- Christian
More information about the hotspot-compiler-dev
mailing list