Request for Review JDK-8006948: Update javac for MethodParameters format change

Kumar Srinivasan kumar.x.srinivasan at oracle.com
Fri Jan 25 10:54:27 PST 2013


> On 01/25/13 13:32, Kumar Srinivasan wrote:
>> Eric,
>>
>>> Hello,
>>>
>>> Please review this change, which updates javac and the classfile library
>>> to reflect changes in the class file format of MethodParameters
>>> attributes.
>> What was the reason to make this change ?
> It was decided that the SYNTHESIZED flag would be 0x8000, not 0x10000 as
> it was in a previous version of the spec.  Therefore, there is no reason
> for the flags field to be u4.
Right I read the discussions on the rationale now.
>
> It bears mention that the u4 causes alignment trap issues if not handled
> carefully.
>
>> Not  true, this will break pack200. and there are  tests in place for this.
>> http://cr.openjdk.java.net/~ksrini/8005252/webrev.1/
>>
> What would it take to update these tests accordingly?  I assume they are
> in TL, and thus can be addressed with a follow-up?

Yes.

>
>> also a spec change for JSR-200 is needed to accommodate this.
>> http://cr.openjdk.java.net/~ksrini/8005252/webrev.pubs.0/webrev/
> Then whoever is in charge of that spec should take note of the changes to
> http://cr.openjdk.java.net/~abuckley/8misc.pdf
I would need to do this, since it is platform spec.

Kumar


>
> Note: I am not in charge of the enhanced metadata spec.  Please consult
> Alex regarding any issues.




More information about the compiler-dev mailing list