[llvm-commits] RFC: TargetRegisterInfo refactoring

Jakob Stoklund Olesen stoklund at 2pi.dk
Tue Nov 9 16:31:09 PST 2010


On Nov 9, 2010, at 3:29 PM, Anton Korobeynikov wrote:

> Hello Everyone,
> 
> Right now TargetRegisterInfo class looks pretty much overweighted and
> definitely not a "register info" anymore. In particular, it contains
> various stuff to handle function stack frames, e.g. emitPrologue /
> emitEpilogue and around. These methods looks much better if located in
> the TargetFrameInfo. I'd like to move them there and collect all
> frame-related stuff in the same place.
> 
> Any objections / comments / thoughts?

That makes sense to me.

It is a lot of code to move around, though. Is it worth the churn?

/jakob




More information about the llvm-commits mailing list