[PATCH] D43962: [GlobalISel][utils] Adding the init version of Instruction Select Testgen

[Amara Emerson via Phabricator via llvm-commits]

aemerson added a comment.

Hi Roman,

This is a very large patch with lots of complexity, and is hard to review. While your description outlines the general idea of the tool, there's little documentation in the code of how this actually works under the hood. Can you add more descriptions so someone can follow what's happening when it comes to maintenance later, e.g. what's LiveInRA supposed to do, what are the pre-conditions and expected outputs of each phase of this tool?

Without that I suspect this patch will continue to lie in the review queue for a long while.

Thanks,
Amara


Repository:
  rL LLVM

https://reviews.llvm.org/D43962