jansvoboda11 added a comment. That's a bit nicer.
Not sure if `RemarkPattern` is a good name now, as it may represent an optimization remark that doesn't have any pattern associated with it. How about calling it `OptimizationRemark` and merging `operator bool` and `operator ->` into `bool patternMatch(...) { return Pattern && Pattern.match(...); }`? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96847/new/ https://reviews.llvm.org/D96847 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits