StaticInst: Merge StaticInst and StaticInstBase.
Review Request #829 - Created Aug. 15, 2011 and submitted
| Information | |
|---|---|
| Gabe Black | |
| gem5 | |
| Reviewers | |
| Default | |
| ali, gblack, nate, stever | |
StaticInst: Merge StaticInst and StaticInstBase. Having two StaticInst classes, one nominally ISA dependent and the other ISA dependent, has not been historically useful and makes the StaticInst class more complicated that it needs to be. This change merges StaticInstBase into StaticInst.
I'm pretty ambivalent about this one... the status quo may be a little overcomplicated, but that never bothered me, and the new version doesn't seem that much simpler (especially since you now have to include a file instead of just forward declaring StaticInstPtr). I'm guessing you have a good reason for doing this though.