[llvm-mca] Move the RegisterFile class into its own translation unit. NFC
ClosedPublic

Authored by mattd on Tue, May 15, 4:33 PM.

Details

Summary

This change will help us turn the DispatchUnit into its own stage.

Diff Detail

Repository
rL LLVM
mattd created this revision.Tue, May 15, 4:33 PM
courbet accepted this revision.Wed, May 16, 1:29 AM

yay, smaller files :)

This revision is now accepted and ready to land.Wed, May 16, 1:29 AM
andreadb accepted this revision.Wed, May 16, 1:33 AM

If the goal is to make the current DispatchUnit a DispatchStage, then this makes sense.

A couple of minor nits. Otherwise it looks good to me.

tools/llvm-mca/Dispatch.h
22 ↗(On Diff #146960)

This include should stay in this file.

tools/llvm-mca/RegisterFile.h
22 ↗(On Diff #146960)

This include file is not needed.
Instead, you should include llvm/MC/MCSchedule.h.

mattd updated this revision to Diff 147112.Wed, May 16, 9:27 AM
mattd marked 2 inline comments as done.

Corrected the #include directives.

mattd added a comment.Wed, May 16, 9:28 AM

Thanks for the reviews.

This revision was automatically updated to reflect the committed changes.