Arch specific changes "platform-specific"

David Simms david.simms at oracle.com
Thu Sep 12 09:18:40 UTC 2019


To aid with current and future porting work...

So normally watching for changes under "src/hotspot/cpu" should be 
enough. I'd like to keep things easy (read "lazy"), and in general it is 
easy to forget special labelling (we are currently using "lworld" and 
"lw<x>" just to avoid using a separate project).

But there are occasions when changes are arch-specific and don't touch 
any of these files. In which case, yes we could be helpful and add a 
label, to point out potential platform issues or attention required.

     E.g. field layout changes might be a good example. Changes happen 
in platform independent code, but often have consequences for different 
platforms.

Proposal:

     Anyone wanting to flag potential platform specific issues simply 
add "platform-specific" (existing label). Best effort, doesn't preclude 
folks making obvious platform specific changes and forgetting the label.

Feel free to come with better ideas (keeping in mind, the least amount 
for "process" is always goal).

Cheers
/David Simms


On 12/09/19 9:56 AM, Dmitry Samersoff wrote:
> Frederic,
>
>> I’ve made the CR x86 specific, and also added [x86] to the summary.
>> Do you need a more specific label?
> What I need is a clear indication that the fix contains platform
> specific code and therefor have to be ported (or validated) to other
> platforms like AArch64.
>
> Label seems to me the most natural way to provide such indication, but
> any other approach will work for me as soon as I can easy select all
> changes for period that requires my attention.
>
> -Dmitry
>
> On 11.09.19 17:13, Frederic Parain wrote:
>> Dmitry,
>>
>> Thank you for the review.
>>
>> I’ve moved the verify_oop() as suggested:
>>
>> http://cr.openjdk.java.net/~fparain/opt_int/webrev.01/index.html
>>
>> I’ve made the CR x86 specific, and also added [x86] to the summary.
>> Do you need a more specific label?
>>
>> Regards,
>>
>> Fred
>>
>>
>>> On Sep 11, 2019, at 09:16, Dmitry Samersoff <dms at samersoff.net> wrote:
>>>
>>> Hello Frederic,
>>>
>>> Looks good to me.
>>>
>>> You may consider to move
>>>
>>> __ verify_oop(rax);
>>>
>>> after done, to save few instructions.
>>>
>>> PS:
>>>
>>> Could you add a label (something like x86_specific) to the CR to
>>> simplify further porting.
>>>
>>> We discussed this approach with IoI and it may be a time to start
>>> adopting it.
>>>
>>> -Dmitry
>>>
>>> On 11.09.19 15:20, Frederic Parain wrote:
>>>> Please this review this small change in the interpreter adding
>>>> a fast path to defaultvalue.
>>>>
>>>> CR: https://bugs.openjdk.java.net/browse/JDK-8230851
>>>> Webrev: http://cr.openjdk.java.net/~fparain/opt_int/webrev.00/index.html
>>>>
>>>> Thank you,
>>>>
>>>> Fred
>>>>




More information about the valhalla-dev mailing list