RFR (S): Issue constants to constant pool in stub code

[Vladimir Ivanov]

Goetz,

+// Inform CodeBuffer that incoming code and relocation will be code
+// Should not be called if start_a_const() returned NULL
+void AbstractAssembler::end_a_const() {
+  end_a_const(code()->insts());
+}

Don't you want to add an assert to check that start_a_const() != NULL?

Best regards,
Vladimir Ivanov

On 11/20/12 1:05 PM, Lindenmaier, Goetz wrote:
> Hi,
>
> When adding constants to the constant pool, the code_section is switched
>
> to the constant pool code_section, and then back again.  It was hard coded
>
> to switch back to the code_section for code.
>
> If constants are issued in stubs, it is switched back to the wrong code
> section.
>
> I changed the code to remember the code section and switch back to the
> proper
>
> one.
>
> Further it’s necessary to relocate only after copying code and stub
> section to
>
> a new location, as the relocation might affect an address in the stub
> section.
>
> You can find these changes here:
>
> http://cr.openjdk.java.net/~goetz/webrevs/webrev-const_in_stub/
>
> or in our ppc port:
>
> http://hg.openjdk.java.net/ppc-aix-port/jdk7u/hotspot/rev/b288e82d9062
>
> Thank you and best regards,
>
> Goetz
>