Granary / granary2

Dynamic binary translation framework for instrumenting x86-64 user space Linux programs
MIT License
39 stars 5 forks source link

Simplify 8_schedule_registers #95

Closed pgoodman closed 9 years ago

pgoodman commented 9 years ago

I think the whole location tracking thing via Loc is overly general / complex. Specifically, all that must be tracked are the arch GPRs, and the single VR being scheduled. It used to be the case with the fragment scheduler (removed a few revisions ago) that multiple VRs were simultaneously scheduled, and so the generality was needed, but now the generality is unnecessary and makes it hard to understand what's really going on.