[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