search

Alamvic / druid

Meta-compiler to generate an optimised JIT compiler frontend based on an Interpreter definition
8 stars 6 forks source link

DRCogitRegisterAllocatorTest vs DRLinearScanRegisterAllocatorTest #130

Open PalumboN opened 5 months ago

PalumboN commented 5 months ago

These two classes are testing the same class: DRCogitLinearScanRegisterAllocator.

Should we merge them?

guillep commented 5 months ago

Hmm, I don't think so. The idea is that the linear scan register allocator has the main algorithm, the cogit one extends it with cogit specific information (interval coallescing and allocation for Division). I'd like to keep them separate... Maybe we need a good comment :)

PalumboN commented 5 months ago

Mmmm, then we should check if the tests are in the correct place.

guillep commented 5 months ago

Yes that too