Skip to content

[luci/pass] Revise Pass to fix gcc-13 error#15241

Merged
seanshpark merged 1 commit into
Samsung:masterfrom
seanshpark:luci_pass_virt
Apr 28, 2025
Merged

[luci/pass] Revise Pass to fix gcc-13 error#15241
seanshpark merged 1 commit into
Samsung:masterfrom
seanshpark:luci_pass_virt

Conversation

@seanshpark

Copy link
Copy Markdown
Contributor

This will revise luci::Pass to fix gcc-13 overloaded-virtual error.

@seanshpark

seanshpark commented Apr 28, 2025

Copy link
Copy Markdown
Contributor Author

@seanshpark seanshpark requested a review from a team April 28, 2025 04:21
shs-park
shs-park previously approved these changes Apr 28, 2025

@shs-park shs-park left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I left some suggestions,
but the LGTM itself.
=)

Comment thread compiler/luci/pass/include/luci/ModulePass.h Outdated
Comment thread compiler/luci/pass/src/VerifyQuantizedNodeType.h Outdated
This will revise luci::Pass to fix gcc-13 overloaded-virtual error.

ONE-DCO-1.0-Signed-off-by: SaeHie Park <saehie.park@gmail.com>

@shs-park shs-park left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you!
👍

@seanshpark seanshpark requested a review from jinevening April 28, 2025 05:44
@seanshpark

Copy link
Copy Markdown
Contributor Author

@jinevening , can you please also take a look?

@jinevening jinevening left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@seanshpark seanshpark merged commit 1c65b56 into Samsung:master Apr 28, 2025
@seanshpark seanshpark deleted the luci_pass_virt branch April 28, 2025 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants