Skip to content

Conversation

llvmbot
Copy link
Member

@llvmbot llvmbot commented Feb 23, 2023

@llvmbot
Copy link
Member Author

llvmbot commented Feb 23, 2023

@AaronBallman What do you think about merging this PR to the release branch?

Copy link
Contributor

@AaronBallman AaronBallman left a comment

Choose a reason for hiding this comment

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

The changes look reasonable to me, but it seems like the release notes are unrelated to the rest of the changes, and I recall accepting a merge to 16.x with this release note before. Did something get mucked up?

@tru
Copy link
Contributor

tru commented Feb 24, 2023

it's just when the release repo gets out of sync with the main repo. i.e. someone pushing release notes directly to the main repo and the release repo doesn't have that commit. it's safe to ignore, we are working on adding automation here.

@tru tru merged commit e3ce92b into release/16.x Feb 24, 2023
@tru tru deleted the llvm-issue57693 branch February 24, 2023 08:42
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.

Bug in the ICF in lld

5 participants