Skip to content

Conversation

@sahithiponangi
Copy link
Contributor

This commit adds assembly file for arraycopy for primitive
nodes in forward and backward directions.

Issue: #6438

Co-authored-by: KONNO Kazuhiro konno@jp.ibm.com

Signed-off-by: Siri Sahithi Ponangi sahithi.ponangi@unb.ca

@knn-k
Copy link
Contributor

knn-k commented Jun 16, 2021

@0xdaryl Could you review this, please? I am a co-author of this PR.

bhi __backwardArrayCopy // byteLength > dstAddr - srcAddr, must do backward array copy
// Forward copy case: fall through

__forwardArrayCopy:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If this is an external callable entry point, please provide some documentation on the API and what it accomplishes functionally.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the comment, but if this is callable from JITed code please describe the API like you do for _arraycopy. For example (tailor it to what the register usage actually is):

// in:    x0 - length in bytes
//        x1 - src addr
//        x2 - dst addr
// trash: x3, x4

finished:
ret

__backwardArrayCopy:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If this is an external callable entry point, please provide some documentation on the API and what it accomplishes functionally.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the comment, but if this is callable from JITed code please describe the API like you do for _arraycopy. For example (tailor it to what the register usage actually is):

// in:    x0 - length in bytes
//        x1 - src addr
//        x2 - dst addr
// trash: x3, x4

.text
.align 2

// Copy array elements
Copy link
Contributor

@0xdaryl 0xdaryl Jun 16, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you provide a bit more of a description of what this code does? For example, this is a generic entry point that will determine which direction to copy as appropriate, so the description of this functionality should reflect that.

This commit adds assembly file for arraycopy for primitive
 nodes in forward and backward directions.

Issue: eclipse-omr#6438

Co-authored-by: KONNO Kazuhiro <konno@jp.ibm.com>

Signed-off-by: Siri Sahithi Ponangi <sahithi.ponangi@unb.ca>
@0xdaryl
Copy link
Contributor

0xdaryl commented Jun 22, 2021

Jenkins build aarch64

@0xdaryl 0xdaryl merged commit c94d049 into eclipse-omr:master Jun 22, 2021
@sahithiponangi sahithiponangi deleted the acasm branch June 22, 2021 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants