This patch makes no assumptions on ABI past the ABI defined in the OpenRISC 1000 spec except that the DWARF register numbers will be 0-31 for registers r0-r31, which is true for both gcc and clang at the moment.
Details
Diff Detail
Event Timeline
Ping? The change seems straightforward to me. It has been extensively tested in our environment as well.
src/Registers.hpp | ||
---|---|---|
1742 | Any particular reason of having 32 variables instead of a single array? |
src/Registers.hpp | ||
---|---|---|
1742 | The PPC registers, on which I based OR1K, were done like that. I will change it to an array. |
src/Registers.hpp | ||
---|---|---|
1779 | Maybe we can rely on the fact that UNW_OR1K_R0 is 1, and therefore instead of switch do the range check on regNum and return _registers.__r[regNum] ? |
src/Registers.hpp | ||
---|---|---|
1785 | Minor nit - space after "if". Same in getRegister(). Ok from me with that change. |
Any particular reason of having 32 variables instead of a single array?