redundant code in Mips arch?

hi,

there are two arrays named DecoderTable32[] and DecoderTable16[] that has no reference to. any idea why they are there? does it make sense to remove them?

(this is llvm 3.3)

thanks,
Jun

forgot to mention: this is from file MipsGenDisassemblerTables.inc

thanks.

Hi,

Those tables are automatically generated by TableGen for each instruction set/instruction set compression. They are related to the Mips16 port that it’s still in development and at the moment lacks direct object emission. That’s the reason the tables aren’t referenced yet.

Regards,

Matheus

Hi Jun,

Are you using the Mips compiler or working on it?

Reed

Hi Jun,

Are you using the Mips compiler or working on it?

yes, i am looking at its code to understand this architecture.

thanks.
Jun

Cool.

So for now, you should not do any commits to that code without getting approval from one of the main developers on it.

There are plenty of things that can be improved and/or cleaned up so the help is welcome but parts of this compiler are already in use in production environments so we have to be careful
not to break things.

Do you have hardware to test with?

LNT will also work with qemu user mode now.

If you actually booted a Mips debian, either with real hardware or qemu, could also run either LNT or test-suite.

I have some other instructions which explain how to run qemu user mode with test-suite.

https://dmz-portal.mips.com/wiki/Building_test-suite_and_running_it_under_Qemu

Reed