Skip to content

Conversation

dcci
Copy link
Member

@dcci dcci commented Feb 20, 2018

And use them in the reflection library (TypeRef). These were
private to TypeRef.cpp but can be moved to the demangler as
they can be of general use, and we can use them from lldb (which
has homemade versions of the functions as well). Bonus point,
it probably makes sense for these helpers to live in the demangler
anyway.

rdar://problem/37710513

And use them in the reflection library (TypeRef). These were
private to `TypeRef.cpp` but can be moved to the demangler as
they can be of general use, and we can use them from lldb (which
has homemade versions of the functions as well). Bonus point,
it probably makes sense for these helpers to live in the demangler
anyway.

<rdar://problem/37710513>
@dcci
Copy link
Member Author

dcci commented Feb 20, 2018

@swift-ci please test

Copy link
Contributor

@eeckstein eeckstein left a comment

Choose a reason for hiding this comment

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

lgtm!

@dcci dcci merged commit c5cdc1b into swiftlang:master Feb 20, 2018
@dcci
Copy link
Member Author

dcci commented Feb 20, 2018

Thanks for the review @eeckstein !

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