aaron.ballman added a comment. In D81336#2090708 <https://reviews.llvm.org/D81336#2090708>, @LegalizeAdulthood wrote:
> Yeah, the restriction to the header file is a bug. It was a misconception on > my part when I originally wrote the matcher. Excellent, thank you for weighing in! I'm fine with the original version of the patch going in with that modification if you'd like to go that route. In D81336#2078669 <https://reviews.llvm.org/D81336#2078669>, @steveire wrote: > This isn't needed if https://reviews.llvm.org/D80961 is merged. Alternatively, I would be fine waiting until that review is merged and making only the main file expansion fixes. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D81336/new/ https://reviews.llvm.org/D81336 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
