mboehme added a comment. In D81392#2088131 <https://reviews.llvm.org/D81392#2088131>, @rsmith wrote:
> I think renaming the flag in the AST dump output would be a good idea, though > it'll be a lot of churn in the tests. I would prefer that we continue to dump > a marker only if the declaration is not unconditionally visible rather than > reversing the sense of the flag in the dump output. Agree -- I had tried that experimentally, but it does cause a lot of noise in the output. > Maybe we should dump the ModuleOwnershipKind in general, not only an > indicator of whether it's Visible or something else? I like this -- though would we then always dump the ModuleOwnershipKind (i.e. causing a lot of churn, as discussed above) or only if it's `VisibleWhenImported` or `ModulePrivate` (i.e. not unconditionally visible)? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D81392/new/ https://reviews.llvm.org/D81392 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits