This revision was automatically updated to reflect the committed changes.
Closed by commit rL330946: Add getDeserializationListener to ASTReader
(authored by yamaguchi, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D45921?vs=143441&id
yamaguchi added a comment.
Do you mean something like `addDeserializationListener` which create a
multiplex listener with an existing listener? I think that'll be good as well!
https://reviews.llvm.org/D45921
___
cfe-commits mailing list
cfe-commit
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
This seems fine, but I wonder if we should expose a mechanism that will create
a multiplex listener automatically as needed?
https://reviews.llvm.org/D45921
___
yamaguchi created this revision.
yamaguchi added reviewers: v.g.vassilev, rsmith, dblaikie, thakis.
We need to know if ASTReader already has a DeserializationListner or
not, and this also helps to create a multiplexing deserialization
listener if there is one already attached.
https://reviews.ll
