[PATCH] D152869: [nfc] Factoring out utility that can be used for other object-level tools
Dayann D'almeida via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 20 14:38:08 PDT 2023
JestrTulip added a comment.
In D152869#4423821 <https://reviews.llvm.org/D152869#4423821>, @jhenderson wrote:
> As there is //currently// only one usage, I don't think a moving of the code benefits anything (you wouldn't do the refactoring if you weren't looking to use it elsewhere, for example). I'm not opposed to the patch being part of a patch series that includes the proposed RFC, but it shouldn't land unless the RFC is accepted and the corresponding patch is still using it once the final version is accepted.
Added the child revision. The RFC is pending review, as it is my initial post. Will update message with link shortly.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D152869/new/
https://reviews.llvm.org/D152869
More information about the llvm-commits
mailing list