[PATCH] Add ARM big endian Target (armeb, thumbeb)

Tim Northover t.p.northover at gmail.com
Thu Mar 20 13:25:44 PDT 2014


  Hi Christian,

  Ah, I see. Possibly not always the whole function, but certainly not just one instruction. Fair enough, it does seem fairly useless.

  Did you notice my other comments (they were inline in Phabricator, at the end of my message on the list)? The tests and the undefined behaviour in particular will need sorting before this can be committed.

  I'm very sorry if you did, and are just updating the patch as you go; I just thought it best to clarify what was going on so we weren't both waiting for the other to respond.

  Cheers.

  Tim.

http://llvm-reviews.chandlerc.com/D3095



More information about the llvm-commits mailing list