[PATCH] D54179: [llvm-mca] Move the AssembleInput logic into its own class.

[Andrea Di Biagio via Phabricator via llvm-commits]

andreadb added a comment.

In https://reviews.llvm.org/D54179#1289783, @mattd wrote:

> I started thinking about another way of organizing this patch, to reduce all of the potential extra files.  I can only really see two CodeRegion generators: one for ASM input, and the other for object file input... once we add the binary support.  Perhaps, we just have a CodeRegionGenerator.{h,cpp} and put the generators for ASM and object files in those instead of having separate header/implementations for ASM and Object files.  Just a  thought.


Sounds reasonable to me.


https://reviews.llvm.org/D54179