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

Tim Northover t.p.northover at gmail.com
Mon Mar 17 23:54:47 PDT 2014


  Hi Christian,

  I only took a small look at the code so fat, but there were some actual code-changes there so we probably want some tests for them. (those fixups and the ARMISelLowering modifications in particular).

  Cheers.

  Tim.


================
Comment at: lib/Target/ARM/MCTargetDesc/ARMAsmBackend.cpp:648
@@ +647,3 @@
+
+  case FK_Data_1:
+  case ARM::fixup_arm_thumb_bcc:
----------------
Why does FK_Data_1 have 2 bytes fixup? If it's not a straight typo, it might be pointing at a bigger abuse in existing code.


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



More information about the llvm-commits mailing list