Skip to content

Conversation

hseok-oh
Copy link
Contributor

@hseok-oh hseok-oh commented Aug 19, 2025

This commit adds shape inference test on Execution test to test shape inference after compilation.
It tests two case: external output buffer and internal output buffer allocation

ONE-DCO-1.0-Signed-off-by: Hyeongseok Oh [email protected]


Draft: #15928

@hseok-oh hseok-oh requested a review from a team August 19, 2025 08:28
@hseok-oh hseok-oh added the PR/ready for review It is ready to review. Please review it. label Aug 19, 2025
@hseok-oh hseok-oh marked this pull request as draft August 19, 2025 10:30
@hseok-oh hseok-oh removed the PR/ready for review It is ready to review. Please review it. label Aug 19, 2025
@hseok-oh

This comment was marked as resolved.

This commit adds shape inference test on Execution test to test shape inference after compilation.
It tests two case: external output buffer and internal output buffer allocation

ONE-DCO-1.0-Signed-off-by: Hyeongseok Oh <[email protected]>
@hseok-oh hseok-oh force-pushed the test_exec_shape_inf branch from df29cf1 to e86800f Compare August 20, 2025 04:44
@hseok-oh hseok-oh marked this pull request as ready for review August 20, 2025 04:44
@hseok-oh hseok-oh added the PR/ready for review It is ready to review. Please review it. label Aug 20, 2025
ragmani
ragmani previously approved these changes Aug 20, 2025
Copy link
Contributor

@ragmani ragmani left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@ragmani ragmani left a comment

Choose a reason for hiding this comment

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

LGTM

@hseok-oh hseok-oh merged commit 8dcde9f into Samsung:master Aug 21, 2025
10 checks passed
@hseok-oh hseok-oh deleted the test_exec_shape_inf branch August 21, 2025 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR/ready for review It is ready to review. Please review it.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants