================ @@ -237,18 +236,6 @@ removeAllUnusedIncludes(llvm::ArrayRef<Diag> UnusedIncludes) { Diag.Fixes.front().Edits.begin(), Diag.Fixes.front().Edits.end()); } - - // TODO(hokein): emit a suitable text for the label. - ChangeAnnotation Annotation = {/*label=*/"", ---------------- hokein wrote:
(I can't comment on the unchanged code), as we don't use ChangeAnnotation for both cases, I think we can remove the annotation-propagation code (line 271-274) in `fixAll`. https://github.com/llvm/llvm-project/pull/76826 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits