ymandel added a comment. In D114809#3165214 <https://reviews.llvm.org/D114809#3165214>, @aaron.ballman wrote:
> LGTM! I had no idea... > > (As a follow-up question, should we be removing the `using` declarations > starting around line 141 or so? Many of them are not used, and if the > documentation dumper can't look through typedefs and gets the documentation > wrong, perhaps we want to discourage using them entirely?) Aaron -- at least in our codebase, those are used a fair amount. What about updating the script to understand those typedefs? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114809/new/ https://reviews.llvm.org/D114809 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
