[PATCH] D74169: [WIP][LLD][ELF][DebugInfo] Skeleton implementation of removing obsolete debug info.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 15 06:28:39 PST 2020


grimar added a comment.

In D74169#1863848 <https://reviews.llvm.org/D74169#1863848>, @avl wrote:

> Creating full set of patches which could be any time applied to the upstream and to the each other  is quite time consuming thing because of time for synchronization/merging/rebasing.


I do not think them needed to be in state `any time applied to the upstream`. `any time` is indeed to much.

Perhaps we could do something like  the following: create a single large patch with all of features which can be applied. And upload it somewhere here as [not for commit].
Then reviewers should be able to review this single patch, but in case of questions they should be able to apply the huge patch to see how the whole feature works.
In this case you'll need to update one huge patch from time to time, e.g. rebase it from time to time after commits of smaller ones. And it should be much simpler.
How does it sound? (the same question for others).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74169/new/

https://reviews.llvm.org/D74169





More information about the llvm-commits mailing list