[ELF] - Detemplate MipsGotSection<ELFT>

Authored by grimar on Mar 17 2017, 10:49 AM.



This continues detemplation process.

Detemplating MipsGotSection<ELFT> is helpfull because can
help to detemplate getRelocTargetVA. (one more change is required)
It opens road to detemplation of GotSection<ELFT> and probably
something else after that.

Diff Detail

grimar created this revision.Mar 17 2017, 10:49 AM
atanasyan accepted this revision.Mar 17 2017, 11:04 AM

LGTM. Thanks.

This revision is now accepted and ready to land.Mar 17 2017, 11:04 AM
ruiu added inline comments.Mar 17 2017, 11:42 AM
170 ↗(On Diff #92168)

This is a sort of unrelated refactoring, so let's not do this in this patch. (Besides size is not a good name.)

214 ↗(On Diff #92168)

I think endianness::little and endianness::bit are silly because there are only two endianness. This should return true or false. Can you rename this isLE and returns if it is a little endian?

ruiu added inline comments.Mar 17 2017, 4:40 PM
170 ↗(On Diff #92168)

I added Config->Wordsize, so please use it.

grimar updated this revision to Diff 92307.Mar 20 2017, 2:54 AM
  • Addressed review comments.
ruiu accepted this revision.Mar 20 2017, 8:08 AM


This revision was automatically updated to reflect the committed changes.