azat added a comment. > If the two patches are basically identical with just some fixes for the > problem, the convention is to reuse the original Differential.
Ok, did not know that, will keep it in mind, thanks! > You can click "Add Action - Open" (or Reopen?), then you can upload a new > patch to update the Differential. Actually I don't have such actions. > This keeps all discussions in one place and keeps are reviewers and > interested folks aware. Yeah, that's a good point, but by some reason I though that it is not possible/good to update already landed revision... P.S. the resubmit version already have some comments too, maybe it will be better to continue there? But I'm not against close it and move it here, if this is preferable and someone can reopen it. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D133092/new/ https://reviews.llvm.org/D133092 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
