Skip to content

Conversation

maarquitos14
Copy link
Contributor

Fix Rule of Three/Five issues reported by static analysis tool.

@maarquitos14
Copy link
Contributor Author

@cyndyishida friendly ping :)

Copy link
Member

@cyndyishida cyndyishida left a comment

Choose a reason for hiding this comment

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

LGTM, but could you change the commit/PR title to be more specific when merging? e.g.
[llvm] Fix Rule of Three/Five issues NFC

@maarquitos14 maarquitos14 changed the title Fix code quality issues. [LLVM][NFC] Fix Rule of Three/Five issues. Sep 29, 2025
@maarquitos14 maarquitos14 merged commit 87bd782 into llvm:main Sep 29, 2025
10 checks passed
mahesh-attarde pushed a commit to mahesh-attarde/llvm-project that referenced this pull request Oct 3, 2025
Fix Rule of Three/Five issues reported by static analysis tool.
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.

2 participants