Skip to content

[dwarfdump] Warn for tags with DW_CHILDREN_yes but no children. #1867

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Oct 7, 2020
Merged

[dwarfdump] Warn for tags with DW_CHILDREN_yes but no children. #1867

merged 2 commits into from
Oct 7, 2020

Conversation

JDevlieghere
Copy link

Flag DIEs that have DW_CHILDREN_yes set in their abbreviation but don't
actually have any children.

rdar://59809554

Differential revision: https://reviews.llvm.org/D88048

(cherry picked from commit e1ef718)

Flag DIEs that have DW_CHILDREN_yes set in their abbreviation but don't
actually have any children.

rdar://59809554

Differential revision: https://reviews.llvm.org/D88048

(cherry picked from commit e1ef718)
@JDevlieghere
Copy link
Author

@swift-ci please test

This test relies on functionality in obj2yaml that's not present on the
stable branch. I made an attempt at backporting all the necessary
changes but quickly realized it was too much churn for this change.
@JDevlieghere
Copy link
Author

@swift-ci please test

@JDevlieghere JDevlieghere merged commit 9ac059c into swiftlang:apple/stable/20200714 Oct 7, 2020
@JDevlieghere JDevlieghere deleted the 🍒/bastille/e1ef7183c6f008fa13cbe273500b020c4fad1252 branch October 7, 2020 23:14
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.

1 participant